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

Encryption app, check config #4268

Merged
merged 4 commits into from
Aug 8, 2013
Merged

Encryption app, check config #4268

merged 4 commits into from
Aug 8, 2013

Conversation

schiessle
Copy link
Contributor

It seems like many users have configuration issues on windows servers with respect to openSSL, see #3629. This pull request catches this error, disables the app again and ask the user to check his server configuration.

cc @karlitschek @DeepDiver1975 @FlorinPeter

@karlitschek
Copy link
Contributor

👍

@ghost
Copy link

ghost commented Aug 1, 2013

Test passed.
Refer to this link for build results: https://ci.owncloud.org/job/pull-request-analyser/299/

@ghost
Copy link

ghost commented Aug 1, 2013

Test failed.
Refer to this link for build results: https://ci.owncloud.org/job/pull-request-analyser/303/

@ghost
Copy link

ghost commented Aug 1, 2013

Test passed.
Refer to this link for build results: https://ci.owncloud.org/job/pull-request-analyser/304/

@ghost
Copy link

ghost commented Aug 1, 2013

Test passed.
Refer to this link for build results: https://ci.owncloud.org/job/pull-request-analyser/307/

@VicDeo
Copy link
Member

VicDeo commented Aug 5, 2013

👍 Successfully failed for me with the log entry

Error Encryption library Missing requirements. Please make sure that PHP 5.3.3 or newer is installed and that OpenSSL together with the PHP extension is enabled and configured properly. For now, the encryption app has been disabled.

openSSL is not configured so I can't test another case.

@VicDeo
Copy link
Member

VicDeo commented Aug 5, 2013

@schiesbn is it in master already?

schiessle added a commit that referenced this pull request Aug 8, 2013
@schiessle schiessle merged commit cc03251 into stable5 Aug 8, 2013
@schiessle
Copy link
Contributor Author

backported to master: 0deae7b...b39d2d1

@schiessle schiessle deleted the encryption_check_config branch August 8, 2013 06:40
@lock lock bot locked as resolved and limited conversation to collaborators Aug 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants