-
Notifications
You must be signed in to change notification settings - Fork 4
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
Version 1.2.0 #57
Version 1.2.0 #57
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
@mstieranka You should now have hopefully all relefant info/keys in your mailbox. Access rights for Chrome Web Store were also updated. Good luck with the release and happy ProgTesting 😉
Try and ask Mr. Vagner for a testing account. He was always very helpful and even added some internal API endpoints used only by this extension. |
1d21f96
to
46337e9
Compare
} | ||
|
||
static getLinksFromHTML(text, href) { | ||
text = text.replace(/<script[^>]*>([\S\s]*?)<\/script>/gim, ""); |
Check failure
Code scanning / CodeQL
Incomplete multi-character sanitization High
<script
} | ||
|
||
static getLinksFromHTML(text, href) { | ||
text = text.replace(/<script[^>]*>([\S\s]*?)<\/script>/gim, ""); |
Check failure
Code scanning / CodeQL
Bad HTML filtering regexp High
elems.forEach((e) => { | ||
const node = document.createElement("a"); | ||
node.href = e.link; | ||
node.innerHTML = `<i>${e.subject}</i> Nová úloha:<br /><b>${e.name}</b>`; |
Check warning
Code scanning / CodeQL
DOM text reinterpreted as HTML Medium
DOM text
DOM text
963f164
to
f2504d8
Compare
0359fd1
to
d868a4c
Compare
Fixes #54 , #56 .
Also includes many new things:
Next steps:
TODO at some later point (maybe next versions):
String.prototype.substr
).js
file, thus loading the page faster)