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.

Sign up for GitHub

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

Jump to bottom

Some JS bugs #194

Open
Saentist opened this issue May 2, 2023 · 1 comment
Open

Some JS bugs #194

Saentist opened this issue May 2, 2023 · 1 comment
Labels
bug good first issue Please feel free to ask me for pointers help wanted

Comments

@Saentist
Copy link

Saentist commented May 2, 2023

javascript-alert
found alert() call; should this be in production code?
web/assets/js/addtableentry.js:7
web/assets/js/addtableentry.js:169
web/assets/js/postxmldoc.js:15

insecure-document-method
User controlled data in methods like innerHTML, outerHTML or document.write is an anti-pattern that can lead to XSS vulnerabilities
web/assets/js/addtableentry.js:30
@Barracuda09 Barracuda09 added bug help wanted good first issue Please feel free to ask me for pointers labels May 3, 2023
@Barracuda09
Copy link
Owner

Hi @Saentist

Thanks for bringing this to my attention.

But I am not a real expert in Web/JS design. I have managed to get this so far on just trying to make it work.
So I would appreciate some help in redesigning and restructuring this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug good first issue Please feel free to ask me for pointers help wanted
Projects
Redesign Web interface
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Barracuda09 @Saentist