8000 TF-3484, 3524, 3717 StallWart interoperability by dab246 · Pull Request #3529 · linagora/tmail-flutter · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

TF-3484, 3524, 3717 StallWart interoperability #3529

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

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

TF-3484, 3524, 3717 StallWart interoperability #3529

wants to merge 4 commits into from

Conversation

dab246
Copy link
Member
@dab246 dab246 commented Mar 4, 2025 •
edited
Loading

Issue

#3484
#3524
#3717

Resolved

demo.webm

@dab246 dab246 requested a review from hoangdat March 4, 2025 04:58
@dab246 dab246 requested a review from tddang-linagora March 4, 2025 04:58
@github-actions GitHub Actions
Copy link
github-actions bot commented Mar 4, 2025

This PR has been deployed to https://linagora.github.io/tmail-flutter/3529.

@chibenwa chibenwa mentioned this pull request Mar 4, 2025
Open
@dab246 dab246 requested a review from tddang-linagora March 4, 2025 08:31
@dscso
Copy link
dscso commented Mar 4, 2025

Does this also work compiling for web?

@dscso
Copy link
dscso commented Mar 4, 2025

I have just tried it on web, but using the standard configuration (env.file) it still uses standard auth.

tddang-linagora
tddang-linagora previously approved these changes Mar 5, 2025
View reviewed changes
@dab246
Copy link
Member Author
dab246 commented Mar 5, 2025 •
edited
Loading

I have just tried it on web, but using the standard configuration (env.file) it still uses standard auth.

IMO, maybe some field configuration incorrect. If possible, please share your env.file with us.

@dscso

Propose

To run on the web with advanced auth (OIDC), in addition to setting the SERVER_URL value, you also need to configure DOMAIN_REDIRECT_URL and WEB_OIDC_CLIENT_ID in the env.file. If you are using Keycloak, go to Your Realm > Clients > Client Details to set these values correctly.

Screenshot 2025-03-06 at 14 02 15

We have also described it in the project's docs folder. Please see more here oidc_configuration.md

dab246 added 4 commits May 19, 2025 15:45
@dab246
TF-3484 Remove GetOIDCIsAvailableInteractor duplicate of `CheckOIDC…
d75c0cc 
…IsAvailableInteractor`

Signed-off-by: dab246 <tdvu@linagora.com>
@dab246
TF-3484 Try get oidc configuration if possible when web finger is not…
5a322cc 
… available

Signed-off-by: dab246 <tdvu@linagora.com>
@dab246
TF-3484 Add unit test to verify get oidc configuration when check oid…
c0c4dbf 
…c available fail

Signed-off-by: dab246 <tdvu@linagora.com>
@dab246
TF-3717 Allow access settings when own email address is null
9581fa1 
Signed-off-by: dab246 <tdvu@linagora.com>
@dab246 dab246 force-pushed the customer-stalwart/tf-3484-stallwart-interoperability branch from 5e2ff75 to 9581fa1 Compare May 19, 2025 09:07
@dab246 dab246 changed the title TF-3484, 3524 StallWart interoperability TF-3484, 3524, 3717 StallWart interoperability May 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tddang-linagora tddang-linagora tddang-linagora left review comments

@hoangdat hoangdat Awaiting requested review from hoangdat

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dab246 @dscso @tddang-linagora
0