Labels Milestones

New Issue

Automaticly generate hash for models #11

Open

opened 4 years ago by Burathar · 0 comments

Burathar commented 4 years ago

Owner

Player.auth_hash and Objective.hash are random fields that should be occupied with a random hash (token_hex(16)) when created. Right now this has to be called explicitly (set_auth_hash() and set_hash() respectively). Because this is prone to programmer error, I'd rather the hash is automaticly generated at creation of the entity.
The db.Column(default=token_hex(16)) method does not work; SqlAlchemy seems to run this function only once for each time the server starts, resulting in duplicate hashes.

Player.auth_hash and Objective.hash are random fields that should be occupied with a random hash (token_hex(16)) when created. Right now this has to be called explicitly (set_auth_hash() and set_hash() respectively). Because this is prone to programmer error, I'd rather the hash is automaticly generated at creation of the entity. The db.Column(default=token_hex(16)) method does not work; SqlAlchemy seems to run this function only once for each time the server starts, resulting in duplicate hashes.

Burathar added the

enhancement-should

label 4 years ago

Burathar referenced this issue 4 years ago

Add Objectives to Game #3

Sign in to join this conversation.

No Branch/Tag Specified

Branches Tags

QR-seachgame

feature_tests

master

testing

Labels

Apply labels

Clear labels

bug
Something is not working duplicate
This issue or pull request already exists enhancement-must
New feature enhancement-should
Non-critical new feature help wanted
Need some help invalid
Something is wrong question
More information is needed security enhancement
New feature/adjustment to improve security wontfix
This won't be fixed

No Label bug duplicate enhancement-must enhancement-should help wanted invalid question security enhancement wontfix

Milestone

Set milestone

Clear milestone

No items

No Milestone

Assignees

Assign users

Clear assignees

No Assignees

1 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Write Preview

Loading…

Cancel

Save

There is no content yet.