fix: incorrect handling of uint8list argument#276
Open
ryan940618 wants to merge 1 commit into
Open
Conversation
ryan940618
marked this pull request as ready for review
September 28, 2025 16:31
ryan940618
changed the title
abc873693
requested changes
Oct 5, 2025
abc873693
left a comment
abc873693
left a comment
Member
There was a problem hiding this comment.
稍微測一下 看起來還是沒解決原本問題
或許是登入時要做相關判斷，要做清除 Cookies 之類動作
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixed the issue that apLoginParser handled uint8list incorrectly.
確保其正確parse html並回傳statusCode
#264 中提到的課表瞬間載入失敗問題可以透過此修改來避免
(此修改促使取得內容失敗時的retry流程順暢)
由於是關鍵組件
尚須測試對於其他功能有無造成問題