github-starred/actual
2
0
Fork 0
mirror of https://github.com/actualbudget/actual.git synced 2026-11-14 21:15:41 -06:00
Code Issues 3.0k Packages Projects Releases 58 Wiki Activity

[GH-ISSUE #2939] [Tech Debt] Incorrect edge version shown in app #50294

New Issue
Open
opened 2026-04-30 13:57:34 -05:00 by GiteaMirror · 3 comments
GiteaMirror commented 2026-04-30 13:57:34 -05:00
Owner
Copy Link
Copy Source

Originally created by @MatissJanis on GitHub (Jun 25, 2024).
Original GitHub issue: https://github.com/actualbudget/actual/issues/2939

We show the active version in the settings page. For demo.actualbudget.org this version number is correct.

For edge: we should show the commit hash there (or other unique identifier).

Screenshot 2024-06-25 at 20 39 39
Originally created by @MatissJanis on GitHub (Jun 25, 2024). Original GitHub issue: https://github.com/actualbudget/actual/issues/2939 We show the active version in the settings page. For `demo.actualbudget.org` this version number is correct. For edge: we should show the commit hash there (or other unique identifier). <img width="1041" alt="Screenshot 2024-06-25 at 20 39 39" src="https://github.com/actualbudget/actual/assets/886567/bcbf37b5-7b1b-4bb2-9f7c-fcadc7b28c04">
GiteaMirror added the tech debthelp wantedgood first issueuser interface labels 2026-04-30 13:57:37 -05:00
GiteaMirror commented 2026-04-30 13:57:39 -05:00
Author
Owner
Copy Link
Copy Source

@matt-fidd commented on GitHub (Jun 25, 2024):

I took a stab at this as one of my first potential changes to actual.

I could either do it through express-actuator (already used for /info /health and /metrics) but it required a git.properties file be generated for every commit and I'm not sure how practical that is.

Alternatively I made a /info/git endpoint that did

execSync('git rev-parse HEAD', { cwd: config.projectRoot }).toString().trim();

Are we okay with exec calls?

<!-- gh-comment-id:2189903545 --> @matt-fidd commented on GitHub (Jun 25, 2024): I took a stab at this as one of my first potential changes to `actual`. I could either do it through `express-actuator` (already used for `/info` `/health` and `/metrics`) but it required a git.properties file be generated for every commit and I'm not sure how practical that is. Alternatively I made a `/info/git` endpoint that did ```js execSync('git rev-parse HEAD', { cwd: config.projectRoot }).toString().trim(); ``` Are we okay with exec calls?
GiteaMirror commented 2026-04-30 13:57:39 -05:00
Author
Owner
Copy Link
Copy Source

@MatissJanis commented on GitHub (Jun 25, 2024):

For actual-server: ideally I wouldn't want to have usage of exec within the codebase. It can potentially lead to arbitrary code execution down the line (all it takes is for a junior to come in and add a variable in the shell command.. and 💣). What if we created a new file when running yarn start that wrote a new file with active commit hash? And then just read from that file to get the hash. Same thing could be applied for yarn build.

For actual: the hash here would be different. No quick ideas how to solve it.

<!-- gh-comment-id:2189919040 --> @MatissJanis commented on GitHub (Jun 25, 2024): For `actual-server`: ideally I wouldn't want to have usage of `exec` within the codebase. It can potentially lead to arbitrary code execution down the line (all it takes is for a junior to come in and add a variable in the shell command.. and 💣). What if we created a new file when running `yarn start` that wrote a new file with active commit hash? And then just read from that file to get the hash. Same thing could be applied for `yarn build`. For `actual`: the hash here would be different. No quick ideas how to solve it.
GiteaMirror commented 2026-04-30 13:57:41 -05:00
Author
Owner
Copy Link
Copy Source

@matt-fidd commented on GitHub (Jun 25, 2024):

Ah sorry! Slipped my mind that this was in actual not actual-server. Generating the file for the server on start is a great idea and would sort that out though!

actual gets its version information directly from package.json, but generating the same file on start and reading from it should do the trick? I'm going to have to think a bit about how builds work and how it would work bundled into actual-server. Maybe if the file isn't there we know it's been imported not directly started and we can fallback to the version in package.json. It wouldn't work for edge builds of actual packaged into actual-server though.

<!-- gh-comment-id:2189934929 --> @matt-fidd commented on GitHub (Jun 25, 2024): Ah sorry! Slipped my mind that this was in `actual` not `actual-server`. Generating the file for the server on start is a great idea and would sort that out though! `actual` gets its version information directly from `package.json`, but generating the same file on start and reading from it should do the trick? I'm going to have to think a bit about how builds work and how it would work bundled into `actual-server`. Maybe if the file isn't there we know it's been imported not directly started and we can fallback to the version in package.json. It wouldn't work for edge builds of actual packaged into `actual-server` though.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
AI generated
API
bank sync
budgeting
bug
can’t replicate
dependencies
docker
documentation
electron
experimental feature
feature
feedback
goal templates
good first issue
help wanted
importers
maintenance
needs info
needs testing
needs triage
needs votes
openid
payees
pull-request
Mirrored from GitHub Pull Request
 regression
reports
responsive
rules
schedules
server
✨ merged
split transactions
tech debt
tech-support
theme
transaction import
transaction reconciliation
transactions
translations
upstream
user interface
✅ approved
wontfix
No labels good first issue help wanted tech debt user interface
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
GiteaMirror ninjasurge (Kaehlen Rog)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#50294