Skip to content

fix(docs): fix broken links in Authorization Endpoint documentation #12749

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

Open
sivasai-662 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(docs): fix broken links in Authorization Endpoint documentation #12749

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions docs/janssen-server/auth-server/endpoints/authorization.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -127,12 +127,12 @@ server for end user authentication.

## Customizing using Interception Scripts

[Interception scripts](../../developer/interception-scripts.md) allows
[Interception scripts](../../../janssen-server/developer/scripts/README.md) allows
flexibility to configure and customize multiple aspects in Janssen Server. For example, see this documentation to learn
how [person authentication](../../developer/scripts/person-authentication.md)
how [person authentication](../../../script-catalog/person_authentication/person-authentication.md)
and [consent gathering](../../developer/scripts/consent-gathering.md) can be
Copy link
Contributor

@coderabbitai coderabbitai bot Nov 25, 2025

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Fix trailing whitespace violation.

Line 133 has trailing spaces detected by markdownlint-cli2 (MD009). Remove the trailing space after "can be".

-and [consent gathering](../../developer/scripts/consent-gathering.md) can be 
+and [consent gathering](../../developer/scripts/consent-gathering.md) can be
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
and [consent gathering](../../developer/scripts/consent-gathering.md) can be
and [consent gathering](../../developer/scripts/consent-gathering.md) can be
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)

133-133: Trailing spaces
Expected: 0 or 2; Actual: 1

(MD009, no-trailing-spaces)

🤖 Prompt for AI Agents 
In docs/janssen-server/auth-server/endpoints/authorization.md around line 133,
there is a trailing whitespace after the word "can be" triggering markdownlint
MD009; remove the extra spaces at the end of that line so it ends with "can be"
(no trailing spaces) and commit the change.

customized using interception scripts.
Comment on lines +130 to 134
Copy link
Contributor

@coderabbitai coderabbitai bot Nov 25, 2025

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

🚨 Critical: Incomplete link fixes — consent gathering link not updated.

Per PR objectives and linked issue #12243, three broken links should be fixed. The Interception scripts (line 130) and person authentication (line 132) links have been correctly updated, but the consent gathering link (line 133) still points to the broken path ../../developer/scripts/consent-gathering.md. According to the issue, it should be updated to ../../../script-catalog/consent_gathering/consent-gathering.md.

Apply this diff to complete the fix:

-[consent gathering](../../developer/scripts/consent-gathering.md) can be 
+[consent gathering](../../../script-catalog/consent_gathering/consent-gathering.md) can be
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
[Interception scripts](../../../janssen-server/developer/scripts/README.md) allows
flexibility to configure and customize multiple aspects in Janssen Server. For example, see this documentation to learn
how [person authentication](../../developer/scripts/person-authentication.md)
how [person authentication](../../../script-catalog/person_authentication/person-authentication.md)
and [consent gathering](../../developer/scripts/consent-gathering.md) can be
customized using interception scripts.
[Interception scripts](../../../janssen-server/developer/scripts/README.md) allows
flexibility to configure and customize multiple aspects in Janssen Server. For example, see this documentation to learn
how [person authentication](../../../script-catalog/person_authentication/person-authentication.md)
and [consent gathering](../../../script-catalog/consent_gathering/consent-gathering.md) can be
customized using interception scripts.
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)

133-133: Trailing spaces
Expected: 0 or 2; Actual: 1

(MD009, no-trailing-spaces)

🤖 Prompt for AI Agents 
In docs/janssen-server/auth-server/endpoints/authorization.md around lines 130
to 134, the consent gathering link is still pointing to
../../developer/scripts/consent-gathering.md; update that link to
../../../script-catalog/consent_gathering/consent-gathering.md so all three
links use the corrected paths per issue #12243, ensuring the Interception
scripts and person authentication links remain as updated.


## Want to contribute?

If you have content you'd like to contribute to this page in the meantime, you can get started with our [Contribution guide](https://docs.jans.io/head/CONTRIBUTING/).
If you have content you'd like to contribute to this page in the meantime, you can get started with our [Contribution guide](https://docs.jans.io/head/CONTRIBUTING/).
Copy link
Contributor

@coderabbitai coderabbitai bot Nov 25, 2025

Choose a reason for hiding this comment

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

🧹 Nitpick | 🔵 Trivial

Consider tightening the language on line 138.

The phrase "in the meantime" may be wordy. Consider a more concise alternative such as: "If you have content you'd like to contribute to this page, you can get started with our [Contribution guide]..."

🧰 Tools
🪛 LanguageTool

[style] ~138-~138: ‘in the meantime’ might be wordy. Consider a shorter alternative.
Context: ...t you'd like to contribute to this page in the meantime, you can get started with our [Contribu...

(EN_WORDINESS_PREMIUM_IN_THE_MEANTIME)

🤖 Prompt for AI Agents 
In docs/janssen-server/auth-server/endpoints/authorization.md around line 138,
the phrase "in the meantime" is wordy; change the sentence to a tighter wording
such as "If you have content you'd like to contribute to this page, you can get
started with our [Contribution guide](https://docs.jans.io/head/CONTRIBUTING/)."
to remove "in the meantime" and keep the same link and meaning.