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

[stable9] Store ext storage creds in pre-login #25530

Closed
wants to merge 1 commit into from

Conversation

PVince81
Copy link
Contributor

  • TODO: evaluate whether this is a viable solution
  • TODO: forward ports

For #25511

@jvillafanez @Xenopathic @DeepDiver1975

@PVince81 PVince81 added this to the 9.0.5 milestone Jul 19, 2016
@mention-bot
Copy link

@PVince81, thanks for your PR! By analyzing the annotation information on this pull request, we identified @Xenopathic, @DeepDiver1975 and @LukasReschke to be potential reviewers

@QuentinC
Copy link

Hello,

I've just applied this PR on " apps/files_external/lib/Lib/Auth/Password/SessionCredentials.php " (I'm on 9.1.0 beta). It doesn't seem to change anything, at least with curl and Android app.. Is there some file elsewhere to change ?

(I don't have the build environment for owncloud..)

@PVince81
Copy link
Contributor Author

@QuentinC hmmm, if it doesn't work in your case it means that 9.1.0 will likely need additional fixing.

Right now I'm trying to find a better solution that is different from this one.

@PVince81
Copy link
Contributor Author

Obsolete and risky

@PVince81 PVince81 closed this Jul 21, 2016
@PVince81 PVince81 deleted the stable9-sessioncredsprelogin branch July 21, 2016 12:59
@lock
Copy link

lock bot commented Aug 5, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants