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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maint cli webserver #769

Draft
wants to merge 1 commit into
base: stable
Choose a base branch
from

Conversation

christopherlam
Copy link
Contributor

@christopherlam christopherlam commented Aug 11, 2020

Gnucash Report Webserver 馃憤

image

@fellen
Copy link
Member

fellen commented Aug 11, 2020

I fear, many users dislike a webserver with their financial data for security reasons.

@christopherlam christopherlam force-pushed the maint-cli-webserver branch 3 times, most recently from f0e8c73 to f75d84e Compare August 11, 2020 15:13
@christopherlam
Copy link
Contributor Author

@jralls
Copy link
Member

jralls commented Aug 11, 2020

I don't think running an http server should be part of GnuCash. It's is an open invitation for remote exploitation since it is trivially easy to create a malicious web page that sends requests to localhost from the browser. I doubt very much that the Guile team puts much thought or effort into the web server's security, but that shouldn't be seen as an opening to go look for a different implementation.

@christopherlam
Copy link
Contributor Author

This is POC for now... May be a private project which uses libgnucash in the future, but can also harness existing report code base.

@christopherlam christopherlam marked this pull request as draft March 3, 2021 16:04
@jralls jralls changed the base branch from maint to stable March 27, 2023 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants