Skip to content

Do not allow JavaScript "eval" in the public share auth page with Talk - #15049

Merged
MorrisJobke merged 1 commit into
masterfrom
do-not-allow-javascript-eval-in-the-public-share-auth-page-with-talk
Apr 11, 2019
Merged

Do not allow JavaScript "eval" in the public share auth page with Talk#15049
MorrisJobke merged 1 commit into
masterfrom
do-not-allow-javascript-eval-in-the-public-share-auth-page-with-talk

Conversation

@danxuliu

Copy link
Copy Markdown
Member

Talk no longer uses JavaScript eval (well, right now it still does, but it will not once the linked pull request is merged), so the Content Security Policy can now be configured to prevent its use in the public share authentication page when the password is protected by Talk.

Talk no longer uses JavaScript "eval", so the Content Security Policy
can now be configured to prevent its use in the public share
authentication page when the password is protected by Talk.
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@danxuliudanxuliu added the 3. to review Waiting for reviews label Apr 11, 2019
@danxuliudanxuliu added this to the Nextcloud 16 milestone Apr 11, 2019
@rullzerrullzer mentioned this pull request Apr 11, 2019
13 tasks

@rullzerrullzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🚀

@rullzerrullzer added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Apr 11, 2019
@MorrisJobke
MorrisJobke merged commit aac22ba into masterApr 11, 2019
@MorrisJobke
MorrisJobke deleted the do-not-allow-javascript-eval-in-the-public-share-auth-page-with-talk branch April 11, 2019 07:43
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@danxuliu@rullzer@MorrisJobke