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

Upgrade jQuery to Version 2 or 3 #73

Closed
AndreKelling opened this issue Mar 16, 2019 · 2 comments · Fixed by #75
Closed

Upgrade jQuery to Version 2 or 3 #73

AndreKelling opened this issue Mar 16, 2019 · 2 comments · Fixed by #75
Assignees

Comments

@AndreKelling
Copy link

AndreKelling commented Mar 16, 2019

Die Version 1 von jQuery birgt Sicherheitslücken.

Ein kurzer Test mit der Version 2 oder 3 lieferte auf dem ersten Blick keine Fehler.
Ich hatte das eben mit den weiter unten stehenden Einbindungen getestet.
Aber ob die neuen Versionen wirklich voll kompatibel mit den verwendeten Skripten sind steht aus.

<script type="text/javascript" src="https://code.jquery.com/jquery-2.2.4.js"></script>
und

<script
  src="https://code.jquery.com/jquery-3.3.1.js"
  integrity_no="sha256-2Kok7MbOyxpgUVvAk/HJ2jigOSYS2auK4Pfzbm7uH60="
  crossorigin="anonymous"></script>

edit:

info:

https://code.jquery.com/jquery/
https://www.willchatham.com/web-dev/a-jquery-1-x-vulnerability-exists-and-no-fix-is-planned/

@polarpixel
Copy link
Member

Vielen Dank für den Hinweis und auch für die Lösungsansätze!

Da wir für die Version 3 prettyPhoto patchen müsste, würden wir lediglich auf die letzte jQuery 2 upgraden. Das werde ich demnächst erledigen.

Es ist ja nur eine Demo ist, die muss mit ihren Assets nicht auf dem aktuellem Stand sein (finde ich), da sie ja nur Dinge zeigt und nicht zum weiteren Arbeiten gedacht ist.

@polarpixel polarpixel self-assigned this May 3, 2019
@AndreKelling
Copy link
Author

ja stimmt schon, aber muss auch nicht schaden.
ich hab es direkt als boilerplate verwendet. wenn man keine projekte mit redaxo gemacht hat, ist das ein sehr guter startpunkt!

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 a pull request may close this issue.

2 participants