Repository navigation
feat: add verified custom API domains - #3801
heathermhuang wants to merge 20 commits into
Conversation
2cc8581 to
59e3cd6
Compare
6a9b9a2 to
03b70cf
Compare
8de38e6 to
154d7a1
Compare
154d7a1 to
5cc55df
Compare
|
All contributors have signed the CLA. ✅ |
|
I have read the CLA Document and I hereby sign the CLA
github-actions[bot] ***@***.***> 于2026年8月12日周三 21:16写道:
… *github-actions[bot]* left a comment (Wei-Shaw/sub2api#3801)
<#3801 (comment)>
Thank you for your contribution! Before we can merge this PR, we need you
all to sign our Contributor License Agreement (CLA)
<https://github.com/Wei-Shaw/sub2api/blob/main/CLA.md>.
*To sign*, please reply with the following comment:
I have read the CLA Document and I hereby sign the CLA
You only need to sign once — it will be valid for all your future
contributions to this project.
------------------------------
I have read the CLA Document and I hereby sign the CLA
------------------------------
*6* out of *7* committers have signed the CLA.
✅ (lenzhang)[https://github.com/lenzhang]
✅ (wucm667)[https://github.com/wucm667]
✅ (Wei-Shaw)[https://github.com/Wei-Shaw]
✅ (fengshao1227)[https://github.com/fengshao1227]
✅ (IanShaw027)[https://github.com/IanShaw027]
✅ (Pluviobyte)[https://github.com/Pluviobyte]
❌ @puppywang <https://github.com/puppywang>
You can retrigger this bot by commenting *recheck* in this Pull Request. Posted
by the *CLA Assistant Lite bot*.
—
Reply to this email directly, view it on GitHub
<#3801?email_source=notifications&email_token=AAVFRBX4MBVD4YGAF2GJMYD5JRU3TA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMRWG4ZTENBYGY42M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-5267324869>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAVFRBTBV64GE6T3BKND2OL5JRU3TAVCNFSNUABGKJSXA33TNF2G64TZHMYTCMJYGYYDCNJRHA5US43TOVSTWNBYGI4DEOBSGYZDPILWAI>
.
Triage notifications, keep track of coding agent tasks and review pull
requests on the go with GitHub Mobile for iOS
<https://github.com/notifications/mobile/ios/AAVFRBXB23JBWA7XHE3HPZD5JRU3TA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMRWG4ZTENBYGY42M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJKTGN5XXIZLSL5UW64Y>
and Android
<https://github.com/notifications/mobile/android/AAVFRBX763KFPH4SDRP4R6T5JRU3TA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMRWG4ZTENBYGY42M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLTGN5XXIZLSL5QW4ZDSN5UWI>.
Download it today!
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
bb36bf3 to
91daee6
Compare
3a22d19 to
8c563a6
Compare
8c563a6 to
d4af7fe
Compare
d4af7fe to
4204979
Compare
|
Update: the administrative-disable finding is fixed in Owner verification preserves administrative disables. Conditional database updates also prevent an in-flight verification from overwriting a later disable or concurrent access changes. Owners cannot delete a disabled record to re-register it; an administrator must explicitly enable or release it. Regression tests cover the original bypass, both DNS outcomes during a concurrent disable, database persistence and explicit re-enable. @Wei-Shaw could you review the updated ownership and lifecycle implementation? Verified inbound domains still add useful white-label and tenant endpoint controls beyond endpoint-display settings. The earlier authorization blocker has been addressed, and the description now documents the proxy/TLS boundary and current verification. |
6cfaf2b to
a4dda7f
Compare
a4dda7f to
fefb358
Compare
fefb358 to
45c7631
Compare
Summary
Add verified inbound custom API domains for user-owned and shared endpoints. Existing endpoint-display settings do not provide ownership verification, per-domain authorization or operator lifecycle controls.
Verification
Current reviewed head:
6cfaf2b5298ad7ff036610e4601e5a768124a934. TLS provisioning and production rollout are separate operational steps.Fixes #3720