Page MenuHomePhorge
Differential D3749

Run rc imap over nginx for 2fa with proxy protocol support
ClosedPublic
Actions

Authored by mollekopf on Jul 28 2022, 5:06 PM.
Tags
None
Referenced Files
F18004790: D3749.id10691.diff
Sat, Jan 11, 8:07 PM
Unknown Object (File)
Mon, Jan 6, 8:04 PM
Unknown Object (File)
Sat, Jan 4, 1:50 PM
Unknown Object (File)
Sun, Dec 22, 12:34 AM
Unknown Object (File)
Thu, Dec 19, 4:14 PM
Unknown Object (File)
Sun, Dec 15, 1:21 PM
Unknown Object (File)
Dec 7 2024, 3:52 PM
Unknown Object (File)
Dec 7 2024, 6:57 AM
View All Files
Subscribers
None

Details

Reviewers
None
Group Reviewers
Restricted Project
Commits
rK8373e36c68f6: Run rc imap over nginx for 2fa with proxy protocol support
rK70ed15887bfa: Run rc imap over nginx for 2fa with proxy protocol support
rK5044e66f9cad: Run rc imap over nginx for 2fa with proxy protocol support
rK15fbcd00cad2: Run rc imap over nginx for 2fa with proxy protocol support
rK903f7774a1f7: Run rc imap over nginx for 2fa with proxy protocol support

Diff Detail

Repository
rK kolab
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mollekopf requested review of this revision.Jul 28 2022, 5:06 PM
mollekopf created this revision.
Harbormaster completed remote builds in B39569: Diff 10688.Jul 28 2022, 5:06 PM
mollekopf added a reviewer: Restricted Project.Jul 28 2022, 5:08 PM
mollekopf updated this revision to Diff 10691.Jul 28 2022, 5:08 PM

Removed log message

Harbormaster completed remote builds in B39572: Diff 10691.Jul 28 2022, 5:08 PM
mollekopf added inline comments.Jul 28 2022, 5:09 PM
src/app/Http/Controllers/API/V4/NGINXController.php
182

Alec pointed out that instead of extracting the Proxy-Protocol-Addr here we could set the Client-Ip in NGINX on the request.

mollekopf added inline comments.Sep 1 2022, 2:31 PM
src/app/Http/Controllers/API/V4/NGINXController.php
182

I have tried this, but don't know how to set the client ip on the http subrequest of the mail module if at all possible.
So we'll stick with this approach until somebody figures out how to do it differently.

This revision was not accepted when it landed; it landed in state Needs Review.Sep 19 2022, 12:11 PM
Closed by commit rK8373e36c68f6: Run rc imap over nginx for 2fa with proxy protocol support (authored by mollekopf). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
docker/
kolab/
utils/
21 lines
nginx/
14 lines
src/
app/
Http/
Controllers/
API/
V4/
10 lines

Diff 10889

View Options

docker/kolab/utils/24-roundcubeconfig.sh

Loading...
View Options

docker/nginx/nginx.conf

Loading...
View Options

src/app/Http/Controllers/API/V4/NGINXController.php

Loading...