Skip to content
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.

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

[17.0][IMP] Update LDAP documentation: disable referral chasing in case of Microsoft AD #11462

Closed
wants to merge 1 commit into from

Conversation

vehi-invitu
Copy link
Contributor

Description of the issue :
We needed to use Microsoft Active Directory (AD) for authentication in Odoo, but the existing documentation didn't address this case. This led to login issues for users even with valid credentials, so i aimed to update the documentation to provide clear instructions on how to disable referral chasing, which is crucial for successful authentication.

cc @cvinh

@robodoo
Copy link
Collaborator

robodoo commented Nov 27, 2024

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team November 27, 2024 00:15
@bram1000
Copy link

bram1000 commented Dec 5, 2024

Reason it wasn't documented: odoo/odoo#86436 (review)
Disabling reference chasing by default: odoo/odoo#189752

So maybe it's a good idea to have this documentation only for 16 to 18 but not in master.

auva-odoo added a commit that referenced this pull request Dec 10, 2024
based on #11462
reformat to comply with doc guidelines
auva-odoo pushed a commit that referenced this pull request Dec 11, 2024
based on #11462
reformat to comply with doc guidelines
@auva-odoo
Copy link
Contributor

auva-odoo commented Dec 11, 2024

@vehi-invitu thank you for your work! I’ve addressed it in PR #11555 and took the opportunity to reformat the page to align with our documentation guidelines.

@auva-odoo auva-odoo closed this Dec 11, 2024
auva-odoo pushed a commit that referenced this pull request Dec 11, 2024
based on #11462
reformat to comply with doc guidelines
auva-odoo pushed a commit that referenced this pull request Dec 11, 2024
based on #11462
reformat to comply with doc guidelines
robodoo pushed a commit that referenced this pull request Dec 11, 2024
based on #11462
reformat to comply with doc guidelines

closes #11555

Signed-off-by: Audrey Vandromme (auva) <[email protected]>
fw-bot pushed a commit that referenced this pull request Dec 11, 2024
based on #11462
reformat to comply with doc guidelines

X-original-commit: 6e9de16
fw-bot pushed a commit that referenced this pull request Dec 11, 2024
based on #11462
reformat to comply with doc guidelines

X-original-commit: 6e9de16
fw-bot pushed a commit that referenced this pull request Dec 11, 2024
based on #11462
reformat to comply with doc guidelines

X-original-commit: 6e9de16
fw-bot pushed a commit that referenced this pull request Dec 11, 2024
based on #11462
reformat to comply with doc guidelines

X-original-commit: 6e9de16
robodoo pushed a commit that referenced this pull request Dec 12, 2024
based on #11462
reformat to comply with doc guidelines

closes #11558

X-original-commit: 6e9de16
Signed-off-by: Audrey Vandromme (auva) <[email protected]>
robodoo pushed a commit that referenced this pull request Dec 12, 2024
based on #11462
reformat to comply with doc guidelines

closes #11556

X-original-commit: 6e9de16
Signed-off-by: Audrey Vandromme (auva) <[email protected]>
robodoo pushed a commit that referenced this pull request Dec 12, 2024
based on #11462
reformat to comply with doc guidelines

closes #11557

X-original-commit: 6e9de16
Signed-off-by: Audrey Vandromme (auva) <[email protected]>
robodoo pushed a commit that referenced this pull request Dec 12, 2024
based on #11462
reformat to comply with doc guidelines

closes #11560

X-original-commit: 6e9de16
Signed-off-by: Audrey Vandromme (auva) <[email protected]>
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.

4 participants