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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request]: SessionID and UUID should be part of all logs #5341

Open
kkmurerwa opened this issue Feb 14, 2024 · 2 comments
Open

[Feature Request]: SessionID and UUID should be part of all logs #5341

kkmurerwa opened this issue Feb 14, 2024 · 2 comments
Labels
enhancement End user-perceivable enhancements. Impact: Medium Moderate perceived user impact (non-blocking bugs and general improvements). Work: Medium The means to find the solution is clear, but it isn't at good-first-issue level yet.

Comments

@kkmurerwa
Copy link
Collaborator

kkmurerwa commented Feb 14, 2024

Is your feature request related to a problem? Please describe.

When merged, #5240 will introduce the ability to log feature flags at the beginning of every app session. Each unique session and feature flag log is identified by a unique UUID and an app session ID.

Currently, a UUID does not exist and will need to be created as per the Core Metrics TDD

Once that is done, in order to track logs and crashes during the session, we will need to add an app session ID and a UUID to each log made on the app.

Describe the solution you'd like

Every log should include an app session ID and a UUID. The two will be created using the modified Twitter Snowflake algorithm as detailed in the Core Metrics TDD. The UUID will help collect logs made in the same session by the same user.

It is important to note that the app session ID is already being populated with a pseudo-random value generated by the LoggingIdentifierController and the generation of this value will need to be switched to the modified Twitter Snowflake algorithm once it has been implemented.

Describe alternatives you've considered

No response

Additional context

No response

@kkmurerwa kkmurerwa added enhancement End user-perceivable enhancements. triage needed labels Feb 14, 2024
@seanlip seanlip changed the title [Feature Request]: SessionID and UUID should part of all logs [Feature Request]: SessionID and UUID should be part of all logs Feb 17, 2024
@adhiamboperes adhiamboperes removed their assignment Mar 12, 2024
@adhiamboperes adhiamboperes added Impact: Medium Moderate perceived user impact (non-blocking bugs and general improvements). Work: Medium The means to find the solution is clear, but it isn't at good-first-issue level yet. labels Mar 12, 2024
@pradyuman04
Copy link

Can I start work on this issue?

@adhiamboperes
Copy link
Collaborator

@pradyuman04, as a first time contributor, it would be great to begin with starter issues from https://github.com/oppia/oppia-android/issues?q=is%3Aissue+is%3Aopen+label%3A%22good+first+issue%22 in order to farmiliarize with the codebase, then you could proceed to something like this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement End user-perceivable enhancements. Impact: Medium Moderate perceived user impact (non-blocking bugs and general improvements). Work: Medium The means to find the solution is clear, but it isn't at good-first-issue level yet.
Development

No branches or pull requests

4 participants