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

Make authentication stance consistent in the document #52

Merged
merged 2 commits into from
Dec 20, 2024

Conversation

achamayou
Copy link
Contributor

@achamayou achamayou commented Dec 4, 2024

Proposed resolution for #14 and #46, as discussed with @SteveLasker, @OR13 and @henkbirkholz.

Will PR the removal of issue statement (the sole endpoint demanding authentication) separately.

@achamayou
Copy link
Contributor Author

@OR13 to your point about interoperability, the document is normative to a limited extent about authentication where present.

@OR13
Copy link
Collaborator

OR13 commented Dec 4, 2024

If we have a lot of "MAY be authenticated" we can consolidate it into a single paragraph, and improve the reading experience for each resource.

@achamayou
Copy link
Contributor Author

If we have a lot of "MAY be authenticated" we can consolidate it into a single paragraph, and improve the reading experience for each resource.

That's what this PR does, I believe, but happy to fix anything I have missed.

Copy link
Contributor

@JAG-UK JAG-UK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@SteveLasker SteveLasker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @achamayou

@JAG-UK JAG-UK linked an issue Dec 5, 2024 that may be closed by this pull request
achamayou added a commit to achamayou/draft-ietf-scitt-scrapi that referenced this pull request Dec 6, 2024
An 'Issue Statement' endpoint is not necessary, nor helpful to implement a transparency service, and it is unclear why it has been added as an optional endpoint.

SCITT services should be able to compose with arbitrary signing services that produce signed statements in the correct format.

Aside from separating concerns and making the document more concise, this helps make the authentication posture consistent (see ietf-wg-scitt#52).
@achamayou
Copy link
Contributor Author

Will PR the removal of issue statement (the sole endpoint demanding authentication) separately.

#53

@SteveLasker SteveLasker merged commit e1ece4c into ietf-wg-scitt:main Dec 20, 2024
1 check passed
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.

Clarify CLIENT AuthZ/AuthN early in doc
4 participants