[Mostly Vibe Coded] Add account info endpoint and improve documentation#6
Merged
WorldObservationLog merged 13 commits intoWorldObservationLog:mainfrom Nov 28, 2025
glomatico:main
Merged
[Mostly Vibe Coded] Add account info endpoint and improve documentation#6WorldObservationLog merged 13 commits intoWorldObservationLog:mainfrom glomatico:main
WorldObservationLog merged 13 commits intoWorldObservationLog:mainfrom
glomatico:main
Conversation
glomatico
commented
Nov 28, 2025
- Add HTTP account info endpoint for getting the storefront, dev token, and music token of the current account.
- Add build from source instructions on README.
- Adjust Dockerfile to use debian:stable-slim instead of Ubuntu.
WorldObservationLog
requested changes
Nov 28, 2025
| static char *g_dev_token = NULL; | ||
| static char *g_music_token = NULL; | ||
|
|
||
| // Forward declarations |
There was a problem hiding this comment.
The code in the forward declarations section is redundant.
| pthread_detach(m3u8_thread); | ||
|
|
||
| pthread_t account_thread; | ||
| struct shared_ptr *ctx_ptr = (struct shared_ptr *)malloc(sizeof(struct shared_ptr)); |
There was a problem hiding this comment.
No need to obtain the ctx_ptr. Just simply pass NULL in pthread_create.
pthread_t account_thread;
pthread_create(&account_thread, NULL, &new_socket_account, NULL);
pthread_detach(account_thread);
Author
|
ok |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.