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

Replace pdf source link #36

Closed
wants to merge 6 commits into from
Closed

Replace pdf source link #36

wants to merge 6 commits into from

Conversation

@srirambv
Copy link
Contributor

srirambv commented Nov 27, 2017

Closes #35

@srirambv srirambv self-assigned this Nov 27, 2017
@srirambv srirambv requested review from kjozwiak, luixxiul and LaurenWags Nov 27, 2017
Copy link
Member

kjozwiak left a comment

@srirambv we should probably either keep the one that's already being listed or use another PDF that's using HTTP... This way we'll have both HTTP & HTTPS PDF's.... Thoughts?

@srirambv
Copy link
Contributor Author

srirambv commented Nov 27, 2017

Sounds good. Added pdf link for both http and https

Copy link
Member

kjozwiak left a comment

@srirambv we should probably also add this change into wikitemplate-android.md as well.

@srirambv
Copy link
Contributor Author

srirambv commented Nov 27, 2017

Added test links for mobile test runs as well

Copy link
Member

kjozwiak left a comment

@srirambv looks like the new HTTP/HTTPS cases for PDF's hasn't been added into wikitemplate-ios.md

@srirambv
Copy link
Contributor Author

srirambv commented Nov 28, 2017

Added the links for iOS too. Should be on all templates now

- [ ] Open a github issue and type some misspellings, make sure they are underlined.
- [ ] Make sure that right clicking on a word with suggestions gives a suggestion and that clicking on the suggestion replaces the text.
- [ ] Open an email on http://mail.google.com/ or inbox.google.com and click on a link. Make sure it works.
- [ ] Test that PDF is loaded over https at https://basicattentiontoken.org/BasicAttentionTokenWhitePaper--pdf

This comment has been minimized.

Copy link
@kjozwiak

kjozwiak Nov 28, 2017

Member

Should be https://basicattentiontoken.org/BasicAttentionTokenWhitePaper-4.pdf

- [ ] Load twitter and click on a tweet so the popup div shows. Click to dismiss and repeat with another div. Make sure it shows
- [ ] Go to https://trac.torproject.org/projects/tor/login and make sure that the password can be saved. Make sure the saved password is auto-populated when you visit the site again.
- [ ] Open an email on http://mail.google.com/ or inbox.google.com and click on a link. Make sure it works
- [ ] Test that PDF is loaded over https at https://basicattentiontoken.org/BasicAttentionTokenWhitePaper--pdf

This comment has been minimized.

Copy link
@kjozwiak

kjozwiak Nov 28, 2017

Member

Should be https://basicattentiontoken.org/BasicAttentionTokenWhitePaper-4.pdf

@srirambv
Copy link
Contributor Author

srirambv commented Nov 29, 2017

Fixed pdf links and updated fingerprint tests for browser-laptop based on brave/browser-laptop#12121 (comment)


**Fingerprinting Tests (Enable fingerprinting protection in `about:preferences#shields`)**
- [ ] Visit https://jsfiddle.net/bkf50r8v/13/, ensure 3 blocked items are listed in shields. Result window should show `got canvas fingerprint 0` and `got webgl fingerprint 00`
- [ ] Visit https://audiofingerprint.openwpm.com/, ensure 2 blocked items are listed in shields. Audio fingerprint graph is shown but doens't contain user fingerprint graph represented in blue line
- [ ] Test that browser is not detected on https://extensions.inrialpes.fr/brave/

This comment has been minimized.

Copy link
@kjozwiak

kjozwiak Dec 5, 2017

Member

This should be moved above the Fingerprinting Tests as it's not related to FP... Test ensures that we're appearing as Chrome.

- [ ] Test that preferences default Bravery settings take effect on pages with no site settings.
- [ ] Test that turning on fingerprinting protection in preferences shows 1 fingerprints blocked at https://browserleaks.com/canvas . Test that turning it off in the Bravery menu shows 0 fingerprints blocked.
- [ ] Test that 3rd party storage results are blank at https://jsfiddle.net/7ke9r14a/7/ when 3rd party cookies are blocked.
- [ ] Test that audio fingerprint is blocked at https://audiofingerprint.openwpm.com/ when fingerprinting protection is on.

This comment has been minimized.

Copy link
@kjozwiak

kjozwiak Dec 5, 2017

Member

Should update the FP tests under iOS to match the formatting & structure of wikitemplate.md.

- [ ] Test that about:preferences default Bravery settings take effect on pages with no site settings.
- [ ] Not Yet Implemented - Test that turning on fingerprinting protection in about:preferences shows 3 fingerprints blocked at https://jsfiddle.net/bkf50r8v/13/. Test that turning it off in the Bravery menu shows 0 fingerprints blocked.
- [ ] Test that 3rd party storage results are blank at https://jsfiddle.net/7ke9r14a/7/ when 3rd party cookies are blocked.
- [ ] Not Yet Implemented - Test that audio fingerprint is blocked at https://audiofingerprint.openwpm.com/ when fingerprinting protection is on.

This comment has been minimized.

Copy link
@kjozwiak

kjozwiak Dec 5, 2017

Member

Should update the FP tests under android to match the formatting & structure of wikitemplate.md.

@srirambv
Copy link
Contributor Author

srirambv commented Dec 6, 2017

Changed mobile template to match laptop template

- [ ] Visit https://jsfiddle.net/bkf50r8v/13/, ensure 3 blocked items are listed in shields. Result window should show `got canvas fingerprint 0` and `got webgl fingerprint 00`
- [ ] Visit https://audiofingerprint.openwpm.com/, ensure 2 blocked items are listed in shields. Audio fingerprint graph is shown but doens't contain user fingerprint graph represented in blue line


This comment has been minimized.

Copy link
@luixxiul

luixxiul Dec 6, 2017

Contributor

nit: remove 1 line.

This comment has been minimized.

Copy link
@srirambv

srirambv Dec 6, 2017

Author Contributor

Fixed

- [ ] Test that PDF is loaded over https at https://basicattentiontoken.org/BasicAttentionTokenWhitePaper-4.pdf
- [ ] Test that PDF is loaded over http at http://www.pdf995.com/samples/pdf.pdf
- [ ] Test that https://mixed-script.badssl.com/ shows up as grey not red (no mixed content scripts are run)
- [ ] Test that https://news.google.com/ sites open in a new tab (due to target being _blank)

This comment has been minimized.

Copy link
@luixxiul

luixxiul Dec 6, 2017

Contributor

mind wrapping _blank with backtick quotes as _ makes something wrapped italic?

- [ ] Test that preferences default Bravery settings take effect on pages with no site settings.
- [ ] Test that 3rd party storage results are blank at https://jsfiddle.net/7ke9r14a/7/ when 3rd party cookies are blocked.

**Fingerprinting Tests (Enable fingerprinting protection in Settings)**

This comment has been minimized.

Copy link
@kjozwiak

kjozwiak Dec 6, 2017

Member

Should the iOS mobile templates use the same new FP cases as wikitemplate.md?

Example: got canvas fingerprint 0 and got webgl fingerprint 00

This comment has been minimized.

Copy link
@srirambv

srirambv Dec 6, 2017

Author Contributor

For iOS there is only Canvas check done so the test url is also different compared to laptop. Can add it back later once security team says its the same result for mobile as well.

This comment has been minimized.

Copy link
@kjozwiak

This comment has been minimized.

Copy link
@diracdeltas

diracdeltas Dec 6, 2017

Member

the changes that @srirambv made to the wiki template recently should probably be reverted since the code change to fingerprinting protection in browser-laptop (brave/browser-laptop#11784) was reverted

- [ ] Test that about:preferences default Bravery settings take effect on pages with no site settings.
- [ ] Test that 3rd party storage results are blank at https://jsfiddle.net/7ke9r14a/7/ when 3rd party cookies are blocked.

**Fingerprinting Tests (Enable fingerprinting protection in Settings)**

This comment has been minimized.

Copy link
@kjozwiak

kjozwiak Dec 6, 2017

Member

Should the android mobile templates use the same new FP cases as wikitemplate.md?

Example: got canvas fingerprint 0 and got webgl fingerprint 00

This comment has been minimized.

Copy link
@srirambv

srirambv Dec 6, 2017

Author Contributor

@diracdeltas could confirm if the expected result is same for laptop and mobile

@srirambv
Copy link
Contributor Author

srirambv commented Dec 13, 2017

Closing in favour of #36

@srirambv srirambv closed this Dec 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.