DB Schema change - Githubissues

moloch-- / RootTheBox

A Game of Hackers (CTF Scoreboard & Game Manager)

http://root-the-box.com/

Apache License 2.0

908 stars 292 forks source link

DB Schema change #601

Closed PJNorrisS1 closed 6 months ago

PJNorrisS1 commented 8 months ago

Seasonal Greetings!

A couple of minor requests if you could be so kind to squeeze in for us please?

.box._description is varchar(4096), however the box description is limited to 1024 - please can that be changed?
add a new field to .flag called _plain_answer as a varchar(256) - this is purely an internal admin db and nothing in the interface. We want to query the DB to show the flags and answers, but not the regex tokens, so we want to store the acceptable answer in this field so we can query it

Thank you PJ

PJNorrisS1 commented 6 months ago

Hello! Any thoughts on the above schema change please?

eljeffeg commented 6 months ago

Sounds good, just haven't had time to look at it. The box description should be an easy fix. The new field will take a little longer as it has a db migration script.

eljeffeg commented 6 months ago

Ok, should be all set.

PJNorrisS1 commented 6 months ago

Many thanks for doing this. qq. should the master build update existing db schemas or is it only when new db's are created? Is the _plain entry visible in the 'create flag' screen, or i manually update the db ?

CyDefender commented 6 months ago

It should update existing db. It is not visible in the UI yet, just the db.

PJNorrisS1 commented 6 months ago

Thanks - didn't update my db, hence asking. I see the box description fixed now. I'll do it again and see how it goes. Thank you.

PJNorrisS1 commented 6 months ago

ah ignore me, picked the wrong db to browse! haha - see _plain_answer now - thank you!