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

Fix authentication problems (fixes #217) #218

Merged
merged 1 commit into from
Sep 16, 2024
Merged

Fix authentication problems (fixes #217) #218

merged 1 commit into from
Sep 16, 2024

Conversation

JimboJoe
Copy link
Contributor

@JimboJoe JimboJoe commented Sep 15, 2024

Problem

  • Authentication through LDAP/oauth is broken

Solution

  • Put back specific upstream patches

PR Status

  • Code finished and ready to be reviewed/tested
  • The fix/enhancement were manually tested (if applicable)

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

@JimboJoe JimboJoe changed the title Fix authentication problems Fix authentication problems (fixes #217) Sep 15, 2024
@JimboJoe
Copy link
Contributor Author

!testme

@yunohost-bot
Copy link
Contributor

📚
Test Badge

@yunohost-bot
Copy link
Contributor

Meow 🐈
Test Badge

@lapineige lapineige merged commit 2dd626e into testing Sep 16, 2024
2 checks passed
lapineige added a commit that referenced this pull request Sep 16, 2024
* Fix hook (#206)

* Auto-update READMEs

* fix hook

---------

Co-authored-by: yunohost-bot <[email protected]>

* Update manifest.toml (#208)

* Update manifest.toml

* Auto-update READMEs

* Add parameters.yml (#209)

* Add parameters.yml

* Update upgrade

* Update change_url

* Function (#210)

* Add parameters.yml

* Update upgrade

* add function

* fix: add time for php script to run a 'long sync' with an application (#213)

* Auto-update READMEs

* [autopatch] Automatic patch attempt for helpers 2.1 (#214)

* Testing (#207)

* Fix hook (#206)

* Auto-update READMEs

* fix hook

---------

Co-authored-by: yunohost-bot <[email protected]>

* Update manifest.toml (#208)

* Update manifest.toml

* Auto-update READMEs

* Add parameters.yml (#209)

* Add parameters.yml

* Update upgrade

* Update change_url

* Function (#210)

* Add parameters.yml

* Update upgrade

* add function

---------

Co-authored-by: Kayou <[email protected]>
Co-authored-by: yunohost-bot <[email protected]>

* Auto-update READMEs

* [autopatch] Automatic patch attempt for helpers 2.1

* cleaning

---------

Co-authored-by: eric_G <[email protected]>
Co-authored-by: Kayou <[email protected]>
Co-authored-by: Yunohost-Bot <>

* Auto-update READMEs

* Update manifest.toml

* Auto-update READMEs

* fix

* Auto-update READMEs

* Update manifest.toml

* Update manifest.toml

* Auto-update READMEs

* cleaning

* Put back upstream patches to fix auth problems (#218)

---------

Co-authored-by: Kayou <[email protected]>
Co-authored-by: yunohost-bot <[email protected]>
Co-authored-by: eric_G <[email protected]>
Co-authored-by: Gofannon <[email protected]>
Co-authored-by: YunoHost Bot <[email protected]>
Co-authored-by: JimboJoe <[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.

3 participants