Skip to content

Conversation

@rdblue
Copy link
Contributor

@rdblue rdblue commented Jan 29, 2022

This adds a 403 response to all of the REST API routes.

Copy link
Contributor

@kbendick kbendick left a comment

Choose a reason for hiding this comment

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

+1. This looks good to me.

Copy link
Member

@RussellSpitzer RussellSpitzer left a comment

Choose a reason for hiding this comment

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

+1

@rdblue rdblue merged commit 27afde6 into apache:master Jan 30, 2022
@rdblue
Copy link
Contributor Author

rdblue commented Jan 30, 2022

Thanks for the reviews!

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.

4 participants