learningequality / learningequality/morango
Remove len>=3 constraint on MORANGO_NODE_ID and MORANGO_SYSTEM_ID
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 15
- Forks
- 23
- PR merge metrics
- No merged PRs in 30d
Description
These envvars are only used when we know we can depend on the Database ID to be unique (when there's no chance the database has been cloned). Requiring certain levels of entropy in these envvars thus doesn't add much -- the true uniqueness will always be coming from the auto-generated (random) Database ID.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No file or test is named in the issue. Start by locating validation for MORANGO_NODE_ID and MORANGO_SYSTEM_ID, then review any existing tests for these environment variables; done means values shorter than three characters are accepted while the remaining behavior is unchanged.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- databases
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100