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

Add warning for disabling email login regarding strict login check #581

Merged
merged 1 commit into from
Aug 1, 2020

Conversation

mrow4a
Copy link
Contributor

@mrow4a mrow4a commented Jun 22, 2020

Adds a warning that this config is not effective if strict checking is not enabled. Requires owncloud/core#37569

image

@codecov
Copy link

codecov bot commented Jun 22, 2020

Codecov Report

Merging #581 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #581   +/-   ##
=========================================
  Coverage     34.79%   34.79%           
  Complexity     1308     1308           
=========================================
  Files            31       31           
  Lines          3719     3719           
=========================================
  Hits           1294     1294           
  Misses         2425     2425           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 950d322...918f74d. Read the comment docs.

@mmattel
Copy link
Contributor

mmattel commented Jul 14, 2020

@pako81 @karakayasemi @HanaGemela please rebase and merge.
Referenced core PR owncloud/core#37569 is already merged.

@HanaGemela can this be part/added to the new upcaoming release?

@karakayasemi
Copy link
Contributor

karakayasemi commented Jul 31, 2020

@phil-davis @mrow4a @micbar I just rebased the PR and CI failed. Also, there are several CI fails in nightly jobs. Looks like, one of the recent changes in core or this app, broke tests. Do you have any idea for the reason?

@phil-davis
Copy link
Contributor

phil-davis commented Jul 31, 2020

@phil-davis @mrow4a @micbar I just rebased the PR and CI failed. Also, there are several CI fails in nightly jobs. Looks like, one of the recent changes in core or this app, broke tests. Do you have any idea for the reason?

You are correct - it got broken with some core acceptance test code changes related to trying to cope with various combinations of LDAP stuff for OCIS testing. When running here in user_ldap it looks like the step:

And user "Alice" has been created with default attributes in the database user backend

now actually ends up creating Alice in LDAP. And then the display name cannot be changed "locally" in oC10.

The problem was not noticed because there is also some problem with pulling osixia/openldap and a lot of CI was failing in the setup steps, even before starting to run any tests. See #590

I will sort out the problem in core...

Update: issue #591 - core PR owncloud/core#37759 has been merged. I an running CI again now to see the result.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants