mirror of
https://github.com/PegaProx/project-pegaprox.git
synced 2026-04-25 10:05:56 +03:00
[GH-ISSUE #75] Why is the actual frontend source code missing? (Potential security and licensing violation) #54
Labels
No labels
Approved
Q2-3 2026 Development
bug
documentation
enhancement
help wanted
invalid
pull-request
question
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/project-pegaprox-PegaProx#54
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @ISCOzmurph on GitHub (Feb 23, 2026).
Original GitHub issue: https://github.com/PegaProx/project-pegaprox/issues/75
Originally assigned to: @mkellermann97 on GitHub.
While reviewing this project, I noticed several major concerns:
Without the real source, it is impossible for anyone to:
This raises a serious question:
Was the omission of the real frontend source intentional (obscuring code), or an oversight?
Given that this application handles privileged credentials for Proxmox clusters, transparency is critical. Please clarify:
If this omission was intentional, the project cannot be considered open source despite the AGPL license. In fact, in its current state, you are in violation of your own included license.
@mkellermann97 commented on GitHub (Feb 23, 2026):
Hi @ISCOzmurph ,
Thanks for raising this – totally understand the concern given the file size.
Out of curiosity, did you actually open the index.html.original file? That's the actual source code. It's not minified or compiled – it's a ~48k line single-file SPA (React 18 + Tailwind).
You can read every line. We chose this structure because it makes deployment extremely simple for our use case (single file copy to update). Unconventional? Absolutely. But it's fully readable, commented, and is the preferred form for modification – so there's no AGPL violation here.
That said, the header comment at line 57 already acknowledges: TODO (NS): Split into proper components when we have time – so a more traditional project structure is on the
roadmap.
Closing this as resolved. Feel free to reopen if anything remains unclear.
Regards,
Marcus
@ISCOzmurph commented on GitHub (Feb 23, 2026):
Thank you for clarifying. However, a 30k+ line single HTML file using in-browser Babel compilation is not a standard or maintainable way to distribute a React application. It makes community contribution nearly impossible. I strongly recommend migrating this to a standard Vite or Create React App structure. You're compliant with the letter of your license but not the spirit; Technically compliant, so that's good. At least I can run a scan to ensure you're not injecting malicious code, based on your word that the "original" is actually what's used in your nebulous internal-only build process. I actually don't have any way of knowing that without an arduous deobfuscation and audit.
I've reevaluated and I do not believe you're in compliance with your attached license.
@mkellermann97 commented on GitHub (Feb 23, 2026):
@ISCOzmurph you're welcome don't worry it's on our internal roadmap and will come out ETA this Sunday. Feel free to share any security findings with us.
Regards,
Marcus