Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Use UUID instead of dateTime for PouchDB/CouchDB _id property #1877

Closed
fox1t opened this issue Mar 3, 2020 · 0 comments 路 Fixed by #1880
Closed

Use UUID instead of dateTime for PouchDB/CouchDB _id property #1877

fox1t opened this issue Mar 3, 2020 · 0 comments 路 Fixed by #1880
Assignees
@jackcmeyer
Labels
馃悰bug issue/pull request that documents/fixes a bug good first issue indicates an issue is good for a first time contributor

Comments

@fox1t
Copy link
Member

fox1t commented Mar 3, 2020

馃悰 Bug Report

Use UUID version 4 to generate cryptographically-strong random values and don't leak any further information. Store it in _id prop of every object that is going to be saved on the DBs.
@fox1t fox1t added the 馃悰bug issue/pull request that documents/fixes a bug label Mar 3, 2020
@fox1t fox1t added the good first issue indicates an issue is good for a first time contributor label Mar 3, 2020
@jackcmeyer jackcmeyer self-assigned this Mar 3, 2020
jackcmeyer added a commit that referenced this issue Mar 4, 2020
@jackcmeyer
fix: use uuidv4 instead of timestamps for autogenerated ids
258cab8 
fix #1877
@jackcmeyer jackcmeyer mentioned this issue Mar 4, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jackcmeyer jackcmeyer

Labels
馃悰bug issue/pull request that documents/fixes a bug good first issue indicates an issue is good for a first time contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Model updates HospitalRun/hospitalrun-frontend
2 participants
@fox1t @jackcmeyer