Skip to content

chore(other): add requiresLogin setting#10

Merged
DmitryAnansky merged 1 commit into
mainfrom
add-requires-login
Jun 15, 2026
Merged

chore(other): add requiresLogin setting#10
DmitryAnansky merged 1 commit into
mainfrom
add-requires-login

Conversation

@DmitryAnansky

Copy link
Copy Markdown
Collaborator

No description provided.

@redocly redocly Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

cafe AI Review: 🟢 Completed

Reunite Agent has reviewed your changes and found 2 potential issue(s).

Note

Low Risk

This is a minor configuration change for Redocly that introduces an access control setting for the documentation, with no direct impact on application code or runtime behavior.

Overview

Updates redocly.yaml to add an access block with requiresLogin: true, which enforces authentication for users attempting to view the hosted API documentation.

Comment thread redocly.yaml
@@ -1,3 +1,5 @@
access:
requiresLogin: true

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Severity: Medium

Requiring login defaults to permitting any Redocly or Google account if sso is not explicitly configured. Consider defining specific identity providers to restrict access to authorized users only.

Comment thread redocly.yaml
@@ -1,3 +1,5 @@
access:
requiresLogin: true

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Severity: Low

When requiresLogin is enabled, consider adding logoutReturnUrl to the access object to ensure users are redirected to a secure, known location after logging out.

@DmitryAnansky DmitryAnansky merged commit b6af132 into main Jun 15, 2026
6 of 7 checks passed
@DmitryAnansky DmitryAnansky deleted the add-requires-login branch June 15, 2026 15:47
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.

2 participants