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
prevent upload from sensitive path #8433
Conversation
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
019db21
to
97d6f29
Compare
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/8433.apk |
CodacyLint
SpotBugs (new)
SpotBugs (master)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM while this also means no real feedback for the user, right?
master-IT test failed, but no output was generated. Maybe a preliminary stage failed. |
master-Unit test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/1354-IT-master-06-16 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:i-trust-you:
/backport to stable-3.16 |
Signed-off-by: tobiasKaminsky tobias@kaminsky.me
Testing
Writing tests is very important. Please try to write some tests for your PR.
If you need help, please do not hesitate to ask in this PR for help.
unit tests
instrumented tests
UI tests