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.

Sign up for GitHub

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

Jump to bottom

Remove optional advisement relying on delegation of authority #357

Merged
merged 1 commit into from
Nov 29, 2021
Merged

Remove optional advisement relying on delegation of authority #357

merged 1 commit into from
Nov 29, 2021

Conversation

csarven
Copy link
Member

@csarven csarven commented Nov 28, 2021 •
edited
Loading

The URI owner may not have delegated authority to the server.

@csarven csarven added doc: Protocol topic: resource access status: Nominated An issue that has been nominated for the next monthly milestone labels Nov 28, 2021
@csarven csarven added this to the Release 0.9 milestone Nov 28, 2021
@csarven csarven mentioned this pull request Nov 28, 2021
Closed
@kjetilk kjetilk self-requested a review November 29, 2021 08:33
kjetilk
kjetilk approved these changes Nov 29, 2021
View reviewed changes
Copy link
Member

@kjetilk kjetilk left a comment

Choose a reason for hiding this comment

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

Indeed, since there is no corresponding server requirement.

@csarven csarven merged commit c5a42bc into main Nov 29, 2021
@csarven csarven deleted the fix/remove-client-check-root-path branch November 30, 2021 09:27
@csarven csarven mentioned this pull request Dec 28, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RubenVerborgh RubenVerborgh RubenVerborgh approved these changes

@kjetilk kjetilk kjetilk approved these changes

Assignees
No one assigned
Labels
doc: Protocol status: Nominated An issue that has been nominated for the next monthly milestone topic: resource access
Projects
Specification
Status: Done
Milestone
Release 0.9
Development

Successfully merging this pull request may close these issues.
3 participants
@csarven @kjetilk @RubenVerborgh