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

8.2.2 auth-token definition duplicate requirements #622

Open
brianjmiller opened this issue Apr 15, 2021 · 3 comments
Open

8.2.2 auth-token definition duplicate requirements #622

brianjmiller opened this issue Apr 15, 2021 · 3 comments

Comments

@brianjmiller
Copy link
Contributor

There are duplicated requirements from the overview in section 8.2.1,

The fetch URL MUST return a JSON structure using a Content-Type of "application/json".

In 8.2.2

The response MUST have a Content-Type of "application/json".

@MrBillMcDonald
Copy link
Contributor

The group agreed that section 8.2 Authorization Token Fetch URL be re-organized to combine the request and possible responses into one section and eliminate duplicate requirement statements. Brian will proposed new wording.

@MrBillMcDonald
Copy link
Contributor

Per the June 24th meeting. The proposed change would be to move the following from 8.2.2 to 8.2.1

The response MUST have a Content-Type of "application/json". The HTTP status code MUST be "200" for a valid request (including one that generates error as described in section 8.2.3).

And then remove this redundant phrase in 8.2.1:

using a Content-Type of "application/json"

@MrBillMcDonald
Copy link
Contributor

MrBillMcDonald commented Feb 16, 2024

Per Feb 16th Meeting - a PR against existing Repo is pending. (including #620)

gavbaa added a commit to gavbaa/CMI-5_Spec_Current that referenced this issue Feb 23, 2024
…ICC#622)

Addresses issue AICC#622, rewords section 8.2.2 to group JSON type details and reorder the sentences from widest to narrowest about the response's contents.
MrBillMcDonald added a commit that referenced this issue Feb 23, 2024
Update 8.2.1/8.2.2 to better order various references to JSON types (#622)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants