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

Develop #28

Merged
merged 104 commits into from
Dec 15, 2024
Merged

Develop #28

merged 104 commits into from
Dec 15, 2024

Conversation

rcmenno
Copy link
Collaborator

@rcmenno rcmenno commented Dec 15, 2024

ref #21 make sure there's one source of truth again regarding development branches. This should prevent unessesary houeskeeping of branches.

rcmenno and others added 30 commits April 17, 2023 11:30
Bumps [werkzeug](https://github.com/pallets/werkzeug) from 2.2.2 to 2.2.3.
- [Release notes](https://github.com/pallets/werkzeug/releases)
- [Changelog](https://github.com/pallets/werkzeug/blob/main/CHANGES.rst)
- [Commits](pallets/werkzeug@2.2.2...2.2.3)

---
updated-dependencies:
- dependency-name: werkzeug
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [flask](https://github.com/pallets/flask) from 2.2.2 to 2.3.2.
- [Release notes](https://github.com/pallets/flask/releases)
- [Changelog](https://github.com/pallets/flask/blob/main/CHANGES.rst)
- [Commits](pallets/flask@2.2.2...2.3.2)

---
updated-dependencies:
- dependency-name: flask
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
…hosting

feat: Add deployment workflow for static files
on-behalf-of: @Azure opensource@microsoft.com
# Conflicts:
#	POC/docs/source/requirements.txt
# Conflicts:
#	README.md
# Conflicts:
#	data/data_template.xlsx
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-stone-02fb10f03-28.westeurope.5.azurestaticapps.net

if (result) {
console.log(result);
document.getElementById("result").textContent = result.text;
window.location = "/entry?code=" + result.text;

Check warning

Code scanning / CodeQL

Exception text reinterpreted as HTML Medium

Exception text
is reinterpreted as HTML without escaping meta-characters.

Copilot Autofix AI 2 months ago

To fix the problem, we need to ensure that the result.text is properly sanitized or escaped before being used in constructing the URL. This can be achieved by using a function to encode the result.text to prevent any malicious scripts from being executed.

The best way to fix the problem without changing existing functionality is to use the encodeURIComponent function to encode the result.text before appending it to the URL. This function encodes special characters, making it safe to include in a URL.

Suggested changeset 1
PWA/public/input_video.html

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/PWA/public/input_video.html b/PWA/public/input_video.html
--- a/PWA/public/input_video.html
+++ b/PWA/public/input_video.html
@@ -74,3 +74,3 @@
                         document.getElementById("result").textContent = result.text;
-                        window.location = "/entry?code=" + result.text;
+                        window.location = "/entry?code=" + encodeURIComponent(result.text);
                     }
EOF
@@ -74,3 +74,3 @@
document.getElementById("result").textContent = result.text;
window.location = "/entry?code=" + result.text;
window.location = "/entry?code=" + encodeURIComponent(result.text);
}
Copilot is powered by AI and may make mistakes. Always verify output.
Positive Feedback
Negative Feedback

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Please select one or more of the options
@rcmenno
Copy link
Collaborator Author

rcmenno commented Dec 15, 2024

@jmargutt @elwinschmitz FYI I will merge this in order to prevent overhead. In the unlikely case that the offline version starts seeing issues because of this, the merge commit can still be reverted.

@rcmenno rcmenno merged commit 912fbc5 into main Dec 15, 2024
7 checks passed
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.

3 participants