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

Show license key as sensitive in report #27

Merged
merged 1 commit into from
May 17, 2017

Conversation

sharidas
Copy link
Contributor

@sharidas sharidas commented May 17, 2017

Show license key as sensitive in the report.

Fixes #26

Signed-off-by: Sujith H sharidasan@owncloud.com

Show license key as sensitive in the report.

Signed-off-by: Sujith H <sharidasan@owncloud.com>
@sharidas sharidas added this to the 10.0.1 milestone May 17, 2017
@PVince81
Copy link
Contributor

Ah I see, so in the past it was purposefully included and now you removed it.

Please post your test result (which you showed me already).

Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 👍

@sharidas
Copy link
Contributor Author

Here is the configreport which I got from the command line occ :
configreport.txt
Here is the configreport downloaded from UI:
configreport_ui.txt

@PVince81 PVince81 merged commit 0404787 into master May 17, 2017
@PVince81 PVince81 deleted the fix-license-key-sensitive branch May 17, 2017 12:21
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.

2 participants