bbs

#159 Implement http for user information and enhance test case

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/100|#100> *:memo: 相關的 issue / Related Issues* • #主 issue <https://github.com/Ptt-official-app/Ptt-backend/issues/52|#52> *:pick: 變更內容 / Details of Changes* • 修改error message • 檢查token permission for real name and money • 補齊testcode

pichuchen 2021-04-02 15:45:27

testcase 的部分應該拆開來比較好

pichuchen 2021-04-02 15:46:07

再來是 readRealname 的permission 設計，應該是在 ISSUE 上面討論再來這樣設計

pichuchen 2021-04-02 15:46:18

在需求上好像沒有這樣設計

gogog22510 2021-04-02 23:51:55

Permission是因為我看到api doc上面有標注，不需要的話我可以拿掉

gogog22510 2021-04-02 23:52:53

test case 拆開的部分可以舉個例子嗎？我原本寫的可能不太好，應該可以直接比對整個data map

pichuchen 2021-04-03 00:13:01

testcase 應該可以參考這樣設計？ https://github.com/Ptt-official-app/Ptt-backend/pull/146/files

pichuchen 2021-04-03 00:13:26

> Permission是因為我看到api doc上面有標注，不需要的話我可以拿掉
是文件上的哪段？

pichuchen 2021-04-03 00:13:27

> Permission是因為我看到api doc上面有標注，不需要的話我可以拿掉
是文件上的哪段？

gogog22510 2021-04-03 23:54:12

在/v1/users/{{user_id}}/information裡有提到需要檢查permission

gogog22510 2021-04-03 23:58:12

參考link裡的test case只有比對map裡的一筆資料，我想要比對所有的field，我可以改成直接比對map本身

#155 [主線] [PTT] 實作轉錄信箱 http module

#152 [主線] [PTT] 實作轉錄看板文章 -- delivery/http

pichuchen 2021-04-09 16:27:34

是參數的那個問題嗎？

Rico 2021-04-17 13:01:05

我現在正在嘗試轉錄看本文章的 repository，我要怎麼測我寫的是可以正常運作的？

Rico 2021-04-17 15:40:26

另外有個問題，回應文章跟轉錄看板文章似乎有點像。
我實際去看 PTT，
回應文章 (y) 的確有分到看板跟信箱；
轉錄 (^X) 似乎只有轉到別的看板而已。

所以實作上，回應文章到看板跟轉錄是一樣的東西囉？

titaneric 2021-04-18 01:07:34

可以hint一下原ptt的檔案位置嗎 方便看

pichuchen 2021-04-18 03:01:46

> 我現在正在嘗試轉錄看本文章的 repository，我要怎麼測我寫的是可以正常運作的？
我覺得可以看一下登入之類的，基本上就是單元測試，所以對應的函式有被呼叫到就算過關？

pichuchen 2021-04-18 03:02:57

> 所以實作上，回應文章到看板跟轉錄是一樣的東西囉？
應該是，但是還是有細節差異，像是他們下的標題以及附加的文字，轉錄對原文章也會產生修改，回應不會。

#169 Feature/#152 forward article

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/152|#152> *:pick: 變更內容 / Details of Changes* *Route* • 在delivery/http/route.go中的`postBoards` method，針對action = "forward_article" 呼叫delivery.ForwardArticle • 在delivery/http裡新增`route_forward_article.go`實作`ForwardArticle` method，呼叫usecase.ForwardArticle • 在usecase/token.go 新增`PermissionForwardArticle` 權限 • 在uscase/usecase.go新增`ForwardArticle` method至Usecase interface • 在usecase/article.go裡面新增`ForwardArticle` method *Test* • 在delivery/http/route_article_test.go裡面新增`forwardArticle` mock test method • 在delivery/http裡新增`route_forward_article_test.go` • 更新swagger.yml action 內容

#170 Feature/#166 取得儲存草稿 - delivery/http

*:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/166|#166> *:pick: 變更內容 / Details of Changes* • 修改 route.go 中的 parseUserPath • 修改 route_users_test.go 中的 TestParseUserPath • 在 route_users.go 中新增 getUserDrafts • 在 route_users_test.go 中新增 TestGetUserDrafts

#182 [一般] 貢獻者名單

*實作細節 / Details of Implement* 因為第一期 milestone 即將結束，目前準備製作一份貢獻者名單加到專案裡面。 包括但不限於 coding，撰寫文件等 歡迎大家提名自己到名單內 *期程 / Schedule* 無，但會定時 merge 到 development *相關文件 / Documents* CONTRIBUTORS.md 提名格式如下: 1. 名字 （網路名稱）：必填 2. GitHub 帳號：非必填 3. 社交帳號：非必填，請給連結 4. 貢獻內容：必填

#76 [一般] 貢獻者名單

*實作細節 / Details of Implement* 目前準備製作一份貢獻者名單加到專案裡面。 包括但不限於 coding，撰寫文件等 歡迎大家提名自己到名單內 *期程 / Schedule* 無，但會定時 merge 到 development *相關文件 / Documents* CONTRIBUTORS.md 1. 名字 （網路名稱）：必填 2. GitHub 帳號：非必填 3. 社交帳號：非必填，請給連結 4. 貢獻內容：必填

#169 Feature/#152 forward article

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/152|#152> *:pick: 變更內容 / Details of Changes* *Route* • 在delivery/http/route.go中的`postBoards` method，針對action = "forward_article" 呼叫delivery.ForwardArticle • 在delivery/http裡新增`route_forward_article.go`實作`ForwardArticle` method，呼叫usecase.ForwardArticle • 在usecase/token.go 新增`PermissionForwardArticle` 權限 • 在uscase/usecase.go新增`ForwardArticle` method至Usecase interface • 在usecase/article.go裡面新增`ForwardArticle` method *Test* • 在delivery/http/route_article_test.go裡面新增`forwardArticle` mock test method • 在delivery/http裡新增`route_forward_article_test.go` • 更新swagger.yml action 內容

#159 Implement http for user information and enhance test case

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/100|#100> *:memo: 相關的 issue / Related Issues* • #主 issue <https://github.com/Ptt-official-app/Ptt-backend/issues/52|#52> *:pick: 變更內容 / Details of Changes* • 修改error message • 補齊testcode

#185 Feature/#166 儲存、刪除草稿 - delivery/http

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/166|#166> *:pick: 變更內容 / Details of Changes* • 在 route_users.go 中新增 postUsers 、postUserDrafts、updateUserDraft、deleteUserDraft • 在 route_users_test.go 中新增 TestUpdateUserDraft、TestDeleteUserDraft • 在 usecase/token.go 中新增 PermissionUpdateDraft、PermissionDeleteDraft

#78 [主線] [PTT] 取得使用者設定

*實作細節 / Details of Implement* • 請見 <https://github.com/ptt/pttbbs/blob/5715b35f510f48eb5092d32882f1aa09181dc3a1/include/pttstruct.h#L57|userec_t> 中的 uflag 和 userlevel *期程 / Schedule* (希望這個任務大概什麼時候做完，如果會卡到其他地方的功能也請在這邊提出。) • 討論時間： 一天 4/19 • 實作時間： 一週 4/25 • 確認時間： 一週 5/2 *相關文件 / Documents* <https://github.com/ptt/pttbbs/blob/5715b35f510f48eb5092d32882f1aa09181dc3a1/include/pttstruct.h#L57|userec_t>

#79 [主線] [PTT] 實作看板發文限制以及細部資訊

*實作細節 / Details of Implement* • <https://github.com/ptt/pttbbs/blob/5715b35f510f48eb5092d32882f1aa09181dc3a1/mbbsd/board.c#L338|board.c> 檔案的 b_config 上面有看板限制和細部資訊，兩邊基本上是一起被取出的，所以可以放在同個 Method 出來。 <https://user-images.githubusercontent.com/600238/115137279-df118b80-a057-11eb-8ded-e1d964caf57d.png|截圖 2021-04-18 下午3 07 59> *期程 / Schedule* (希望這個任務大概什麼時候做完，如果會卡到其他地方的功能也請在這邊提出。) • 討論時間： 一天 4/19 • 實作時間： 一週 4/26 • 確認時間： 一週 5/3 *相關文件 / Documents*

#187 implement GetPopularBoards usecase and add mock test in board_mock_test

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/164|#164> • close <https://github.com/Ptt-official-app/Ptt-backend/issues/181|#181> *:memo: 相關的 issue / Related Issues* • 主issue <https://github.com/Ptt-official-app/Ptt-backend/issues/51|#51> *:pick: 變更內容 / Details of Changes* • usecase 1. 新增usecase method:GetPopularBoards 2. 新增usecase test:TestGetPopularBoards • <https://github.com/Ptt-official-app/Ptt-backend/issues/181|#181> : 由於直接使用GetBoards取得資料再由usecase進行篩選，所以沒有新增function

iThome

Codecov被駭，殃及數百家企業客戶及IBM等科技公司

目前美國政府調查發現，數百家Codecov客戶可能已被竊走內部軟體開發專案資料，資料外洩規模超出Codecov原先預期的範圍

andylee1013 2021-04-22 19:59:41

有喔 有些PR有看到report
不過又看到熟悉的關鍵字Solarwinds 😂

#189 add forwardArticleToEmail

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/156|#156> *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/156|#156> • <https://github.com/Ptt-official-app/Ptt-backend/issues/67|#67> *:pick: 變更內容 / Details of Changes* • 在 config_default.toml 新增 mailDriver 設定，預設為 smtp • 在 usecase.NewUsecase 會依據 config.MailDriver 設定 mail • 在 usecase 新增 updateMail 替換不同的 mail • 新增 mail.go 定義 Mail interface

#175 [主線] [PTT] 增加 route_borad 相關註解

*實作細節 / Details of Implement* 主要是增加 route_borad 相關註解 可參考 <https://github.com/Ptt-official-app/Ptt-backend/issues/110|#110> *期程 / Schedule* • 討論時間： 三天 • 實作時間：一天 • 確認時間： 一天 *相關文件 / Documents* route_boards.go, route_boards_test.go , route_boards_mock_test.go

pichuchen 2021-04-24 02:33:40

guest 的話應該還是要以 guest 然後無密碼去取得 token?

Chang Chen Chien 2021-04-24 16:30:35

恩對，因為在我的理解裡，guest 也是要能拿到 board list 的，而不是 得到 `token_invalid` 就什麼都看不到>

Chang Chen Chien 2021-04-24 16:30:45

不知道這樣理解是否正確？

pichuchen 2021-04-25 18:49:36

不太正確，因為不是所有 BBS 站 guest 都能拿到 board list, 是最近幾年 ptt 出現網頁版，所以才會拿到。

另外不同權限的人拿到的 board list 都會有點不同，會有些隱藏看板看不到。

Chang Chen Chien 2021-04-26 08:31:01

了解了， 反正我先送 PR 給大家 review ，在 github g上再討論哪邊需要修正

#192 Feature/#168 儲存草稿 - usecase

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/168|#168> *:pick: 變更內容 / Details of Changes* • 在 usecase/user.go 中新增 GetUserDrafts 、UpdateUserDraft、DeleteUserDraft、isValidDraftID（目前合法的 draftID 為 0~9） • 在 usecase/user_test.go 中新增 TestGetUserDrafts、TestUpdateUserDrafts

#193 新增 Dummy Logger for Testing

*:clap: 解決掉的 issue / Resolved Issues* *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/71|#71> *:pick: 變更內容 / Details of Changes* • `logging` • 新增 `DummyLogger` 結構，method 皆為空函式 • `main` • 修改 `NewRepository` `NewUsecase` `NewHTTPDelivery` 傳入變數 `logger` 作為相依 _(Dependency)_ • `http` • 修改測試函式皆使用 `DummyLogger` 為相依 • `repository` • 修改測試函式皆使用 `DummyLogger` 為相依 • `usecase` • 修改測試函式皆使用 `DummyLogger` 為相依 _注意：此PR目的是演示一下 mock logger 的概念_

andylee1013 2021-04-25 16:52:43

自己回復一下~關於mock logger 的做法有很多種，另一個做法是用 mockgen 產生一個實作，但是在Testing code裡面就會每個test case都要去Expect 所有的logger method。

pichuchen 2021-04-25 16:54:43

我是覺得他可以丟到 global 去這樣

andylee1013 2021-04-25 17:01:25

所謂的global是指什麼

andylee1013 2021-04-25 17:03:10

我有想過 mock_logger 裡面放一個 var xx = &DummyLogger{} 就給所有test case用

pichuchen 2021-04-25 18:23:50

> 我有想過 mock_logger 裡面放一個 var xx Logger = &DummyLogger{} 就給所有test case用
類似這樣，可能就是有個 defaultLogger 像是 http.defaultClient 這樣

andylee1013 2021-04-25 18:24:36

恩恩 其實實際的logger 跟DummyLogger 都可做這件事

andylee1013 2021-04-25 18:25:02

我覺得只要大家有個共識就可以

#190 Implement get user info usecase

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/165|#165> *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/52|#52> *:pick: 變更內容 / Details of Changes* • repository • 新增 `GetUserInformation()` • 注意 function 尚未實作 • usecase • 修改 `GetUserInformation()` 回傳 repository 的 return 值 • 新增 `TestGetUserInformation()`

ocf.tw

PTT ID Profiling | 財團法人開放文化基金會 (OCF)

網路上的不實資訊、網軍操作帶風向的狀況層出不窮，批踢踢實業坊 (PTT) 的政治類看板, 比起其他看板，帶風向的情形更加嚴重，網路上已經有很多專家利用爬蟲工具，自行搜尋分析出活動異常的 PPT 使用者帳號。本計畫希望能網羅各方資源之外，也能開發出套完整的開源工具，讓使用者自行析判斷誰是誰非。

devptt.site

React App

Web site created using create-react-app

devptt.site

React App

Web site created using create-react-app

devptt.site

React App

Web site created using create-react-app

yushuanhsieh.github.io

Switch to a register-based calling convention for Go functions

自己對於 memory layout 相關議題都蠻感興趣的，而這次 Go 1.17 有一項效能改善的 proposal: switch to a register-based calling convention for Go functions 剛好跟 memory 有相關，因此就看了一下 proposal 文件介紹，不但複習了在計算機架構中曾接觸到的 calling convention 知識，也對於 Go 內部機制有更多認識。

pichuchen 2021-04-26 01:19:51

更深的東西在這邊 https://go.googlesource.com/proposal/+/refs/changes/78/248178/1/design/40724-register-calling.md

#195 增加 route_boards 相關註解

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/175|#175> *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/109|#109> • <https://github.com/Ptt-official-app/Ptt-backend/issues/110|#110> *:pick: 變更內容 / Details of Changes* • 新增註解 • 新增 handler 的 Request URL 與 API Doc link • 對還沒列入 swagger 的 API Doc link 加上 TODO prefix

#154 [主線] [PTT] 實作轉錄看板文章 -- repository

*實作細節 / Details of Implement* • 此為 <https://github.com/Ptt-official-app/Ptt-backend/issues/66|#66> 子任務 • 實作 <https://github.com/Ptt-official-app/Ptt-backend/blob/development/internal/repository/repository.go|repository.go> • 因為 <https://github.com/Ptt-official-app/go-bbs|go-bbs> 尚未完成，需要先 mock 一個暫時性的結果等待 go-bbs 完成之後進行串接。 *期程 / Schedule* • 討論時間： 一週, 到 4/7 • 實作時間：一週, 到 4/14 • 確認時間：一週, 到 4/21 *相關文件 / Documents* • 主issue: <https://github.com/Ptt-official-app/Ptt-backend/issues/66|#66> • <https://docs.google.com/document/d/18DsZOyrlr5BIl2kKxZH7P2QxFLG02xL2SO0PzVHVY3k/edit|PTT後端系統協定>

internal/usecase/article.go:21

``` //result, err := usecase.repo.AppendComment(ctx, userID, boardID, filename, appendType, text) ```

#200 根據 whitefloor 調整了既有的說明文檔

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/151|#151> *:pick: 變更內容 / Details of Changes* 根據 whitefloor 調整了一些文檔上的問題 主要的更動如下: 1. 將歷史緣由跟開發原因搬進 FOR_DEVELOPERS.md 裡面 我覺得作為一個專案的門面，應當盡量簡單扼要說明這個專案的的功能。 2. 直接增加了如何參與這個專案 由於希望此專案不要變成一人開發，需要用更簡單的方法吸引人進來參與 3. 增加了 Q&amp;A 我將 Q&amp;A 拆成了兩個 part ，一個是給要深入專案的，一個是給第一次參與的

#198 增加 smtp provider 回傳

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/196|#196> *:memo: 相關的 issue / Related Issues* • <https://github.com/Ptt-official-app/Ptt-backend/issues/191|#191> • #0 *:pick: 變更內容 / Details of Changes* 增加 smtp provider

#80 Feature/#79 看板發文限制以及細部資訊

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/go-bbs/issues/79|#79> *:pick: 變更內容 / Details of Changes* • 在 pttbbs/board.go 中新增 InfoAndSettings • 在 pttbbs/board_test.go 中新增 TestInfoAndSettings

#82 [主線] [PTT] 實作在文章底部新增一行文字的method

*實作細節 / Details of Implement* • 目前在 <https://github.com/Ptt-official-app/Ptt-backend/issues/194|實作文章推文> 以及 <https://github.com/Ptt-official-app/Ptt-backend/issues/66|轉錄看板文章> 都需要直接新增一行字串 • 直接編輯整個文章和在後面新增一行字串在硬碟上的效率不太一樣，新增字串會有比較高的效能 • 可能會在原有文章或是 bbs 的 connecter 新增一個 method 裡面有參數來表示某個看板的某個文章，然後和一行任意文字。 *期程 / Schedule* (希望這個任務大概什麼時候做完，如果會卡到其他地方的功能也請在這邊提出。) • 討論時間： 三天，5/2 • 實作時間： 一週，5/9 • 確認時間： 一週，5/16 *相關文件 / Documents* 基本上是為了實作這樣的功能 <https://user-images.githubusercontent.com/600238/116584922-71fcd080-a94a-11eb-9e6b-7eb0c068e71b.png|截圖 2021-04-30 上午12 24 12>

#202 推文repo更新

*:clap: 解決掉的 issue / Resolved Issues* • close <https://github.com/Ptt-official-app/Ptt-backend/issues/194|#194> *:pick: 變更內容 / Details of Changes* • `AppendComment`統一回傳`PushRecord` interface，這需要在go-bbs更新 • 改寫`AppendComment` response body in delivery，使其與文件一致 • 測試`AppendComment` repo mocked method