Skip to content

docs: Gitlab Pipelines federated OIDC client authentication#239

Open
J4NS-R wants to merge 4 commits intopocket-id:mainfrom
J4NS-R:patch-1
Open

docs: Gitlab Pipelines federated OIDC client authentication#239
J4NS-R wants to merge 4 commits intopocket-id:mainfrom
J4NS-R:patch-1

Conversation

@J4NS-R
Copy link

@J4NS-R J4NS-R commented Mar 14, 2026

Added Gitlab Pipelines section for federated OIDC client authentication with Pocket ID, including configuration values and an example job.

Added Gitlab Pipelines section for OIDC client authentication with Pocket ID, including configuration values and an example job.
@J4NS-R J4NS-R changed the title Document Gitlab Pipelines OIDC client authentication docs: Gitlab Pipelines OIDC client authentication Mar 14, 2026
@J4NS-R J4NS-R changed the title docs: Gitlab Pipelines OIDC client authentication docs: Gitlab Pipelines federated OIDC client authentication Mar 14, 2026
@J4NS-R J4NS-R marked this pull request as ready for review March 14, 2026 09:52
Copy link
Contributor

@ItalyPaleAle ItalyPaleAle left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM with some suggestions

- apk add --no-cache curl jq
- responsefile=$(mktemp)
- |
curl -SsfX POST --url "https://pocketid.example.com/api/oidc/token" \
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggestion: make the endpoint a variable?

@github-actions
Copy link
Contributor

github-actions bot commented Mar 14, 2026

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
pocket-id-website ✅ Ready (View Log) Visit Preview 2ef7de6

J4NS-R and others added 2 commits March 14, 2026 11:07
Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
@J4NS-R
Copy link
Author

J4NS-R commented Mar 14, 2026

@ItalyPaleAle thanks for the review. Review changes made 👍

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