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

[INTEGRATION] Dolibarr update #831

Open
2 tasks
olaxe opened this issue Feb 4, 2024 · 3 comments
Open
2 tasks

[INTEGRATION] Dolibarr update #831

olaxe opened this issue Feb 4, 2024 · 3 comments
Labels
documentation Improvements or additions to documentation integration Connection between LLDAP and a service question Further information is requested

Comments

@olaxe
Copy link

olaxe commented Feb 4, 2024

Checklist

  • Check if there is already an example config for it.
  • Try to figure out the configuration values for the new service yourself.
    • You can use other example configs for inspiration.
    • If you're having trouble, you can ask on Discord or create an issue.
    • If you succeed, make sure to contribute an example configuration, or a configuration guide.
  • If you hit a block because of an unimplemented feature, create an issue.

Description of the service
Add this information to the Dolibarr service as it is buggy for the moment: Dolibarr/dolibarr#27993

What you've tried
It is working as it is if you have only one user in the Dolibarr LLDAP group. If not, only the first user of the group will be authenticated. Other will generate error 500 and empty webpage. When your user has been replicated into Dolibarr, you can authenticate even if you have been removed from the Dolibarr LLDAP group. Sometimes, when there is an error, you will have an empty webpage and generate an error 500.

My patches to the Dolibarr code fix all of these behavior.

@olaxe olaxe added the integration Connection between LLDAP and a service label Feb 4, 2024
@hregis
Copy link

hregis commented Feb 4, 2024

@olaxe i don't understand your problem with the group !

@olaxe
Copy link
Author

olaxe commented Feb 5, 2024

The problem is with the Dolibarr code. Everything is working at expected with LLDAP. I just propose an update of the Dolibarr integration to warn other LLDAP users.

@nitnelave
Copy link
Member

@olaxe since you're the one with the most context, could you send a PR with the proposed update to the dolibarr code?

@nitnelave nitnelave added documentation Improvements or additions to documentation question Further information is requested labels Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation integration Connection between LLDAP and a service question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants