Skip to content

feat: Add support for forward auth (Rebased)#1470

Closed
nanospearing wants to merge 16 commits into
seerr-team:developfrom
nanospearing:fw-auth-rebase
Closed

feat: Add support for forward auth (Rebased)#1470
nanospearing wants to merge 16 commits into
seerr-team:developfrom
nanospearing:fw-auth-rebase

Conversation

@nanospearing

Copy link
Copy Markdown

Original Description from #1341:

Description

Adds support for automatically authenticating as the user specified in the X-Forwarded-User header when the setting enableForwardAuth is set.

This is basically the same as #580 (the first 2 commits are cherry picked from that PR) but rebased and with suggestions applied

Screenshot (if UI-related)

image #### To-Dos * [x] Successful build `pnpm build` * [x] Translation keys `pnpm i18n:extract`

Issues Fixed or Closed


This PR is a rebased version since the original developer hasn't done it yet :P

@fallenbagel

Copy link
Copy Markdown
Member

@nanospearing please don't merge develop and instead rebase as told on the contributing.md

Comment thread Dockerfile
;; \
esac

RUN npm install --global pnpm@9

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why is this being changed in this pr

Comment thread Dockerfile

RUN apk add --no-cache tzdata tini && rm -rf /tmp/*

RUN npm install -g pnpm@9

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Same here.

@nanospearing nanospearing deleted the fw-auth-rebase branch March 16, 2025 19:46
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.

Support for Forward Auth

4 participants