Repository navigation
fix(qoder): stop derived machine headers from hiding daily check-in - #487
Merged
Merged
Conversation
Campaign requests sent a full derived cosy-machine set, which upstream filters as a non-official client, so the daily credit campaign vanished after the first claim. Send only the desktop identity headers, retry an empty campaign list, and treat SAME_PERSON_ALREADY_CLAIMED as already checked in.
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
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.
Summary
Cosy-Machine*set,Origin, orReferer. Upstream treats a full derived set as a non-official client and drops the daily credit campaign from an otherwise successful list, which is why check-in succeeded once and then reported no campaign (Qoder 国内版签到：每个账号登录后仅能签到一次，之后手动/自动签到均无效（PAT 与浏览器登录，v0.7.0 仍复现） #323).SAME_PERSON_ALREADY_CLAIMED(person-level dedupe, including when the official client already collected) is recorded as already checked in today instead of a failure.Chat fingerprints are unchanged (
Cosy-Versionstays1.1.75). The desktop campaign version stays0.4.8. Nativeruntime-infoidentity is not added; without it the Global daily credit may still be absent, but the gateway no longer hides it.Test plan
go test ./internal/providers/qoder/ -count=1 -run 'Checkin|Campaign'