Skip to content

Conversation

@Faraz32123
Copy link
Contributor

@Faraz32123 Faraz32123 commented May 2, 2025

- It's related previous PR: https://github.com/openedx/edx-platform/pull/36643/files.
- This time followed LMS template(https://github.com/openedx/edx-platform/blob/5b3caa93e218e38e7459bdc1ff99ee175e93a22b/lms/templates/courseware/courseware-chromeless.html#L126C7-L127C42) as we are not getting any error on LMS.

Attaching error for context:
`Uncaught SecurityError: Failed to read a named property 'offsetHeight' from 'Window': Blocked a frame with origin "https://studio.stage.edx.org/" from accessing a cross-origin frame.`
@Faraz32123 Faraz32123 force-pushed the fix/security_issue branch from b77dc15 to 9ea9007 Compare May 2, 2025 07:42
@Faraz32123 Faraz32123 requested a review from navinkarkera May 2, 2025 08:25
@Faraz32123 Faraz32123 marked this pull request as ready for review May 2, 2025 13:22
@rayzhou-bit
Copy link
Contributor

@navinkarkera Could you please review again? I think merging is blocked due to requested changes.

@Faraz32123 Faraz32123 merged commit be5a625 into master May 6, 2025
49 checks passed
@Faraz32123 Faraz32123 deleted the fix/security_issue branch May 6, 2025 06:13
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants