Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CDN: Field Update & Bug Fix #79

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

BenBeng
Copy link

@BenBeng BenBeng commented Sep 2, 2022

  1. OriginPeer add fields Weight,ISP.
  2. RequestAuth add field TimestampMetric.
  3. Fix: When TrafficLimit.LimitStartHour set to 0, console will show null.
  4. Fix: RefererACL, IpACL and UaACL can't clear blacklist or whitelist.
  5. Fix: RequestAuth can't be disabled.

1. OriginPeer add fields `Weight`,`ISP`.
2. RequestAuth add field `TimestampMetric`.
3. Fix: RefererACL, IpACL and UaACL can't clear blacklist or whitelist.
4. Fix: When TrafficLimit.LimitStartHour set to 0, console will show null.
5. Fix: RequestAuth can't be disabled.
To match corresponding field type in the response of API `GET /v2/domain/{domain}/config`
@BenBeng
Copy link
Author

BenBeng commented Sep 5, 2022

Change RequestAuth.TimestampMetric type to int to match corresponding field type in the response of API GET /v2/domain/{domain}/config.

@BenBeng
Copy link
Author

BenBeng commented Sep 5, 2022

According to this documentation, timestampMetric is a string. Setting this field to number 10 or 16 also works. But setting yyyyMMDDhhmm to the field actually results in a setting of 10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant