-
Notifications
You must be signed in to change notification settings - Fork 15
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
Adding the Admin User during 'CREATE Users & Teams' fails due to lack of T_TYPE #10
Comments
Yes you are correct line 1014 should include T_TYPE. There were some events where we distributed tokens to the users and asked them to register via homepage ( Users Register using tokens : I haven't noticed this bug after adding the new feature, I will label this issue as a bug and will fix in the later update. However, this project is merged with my new CTF platform project (which is under development : https://github.com/rgajendran/ctf_platform) which is integrated with oVirt Virtualisation, SecGen, CTF Marker and custom created platform. |
Just pulled ctf_platfrom, same issue. What version of mysql are you using? |
I use xampp for this project ( I just tested the marker on xampp, it works perfectly for me. I don't recommended you to use |
I double checked it, it works on my Fedora aswell. Incase if you haven't used XAMPP, follow these steps.
I hope it works for you, let me know if you have any issues. |
Got it working when using XAMPP vs manual apache2 and MySQL. Looks like MySQL and MariaDB don't behave the same way. Might be helpful to put that XAMPP is the approved hosting solution in the README.md For reference, this is the version of MariaDB that works for me: Thanks for troubleshooting with me. |
This was very helpful. Thank you both. |
ctf_marker/admin.php
Line 1014 in 5135484
The users DB is defined with
`T_TYPE` varchar(1) NOT NULL,
But the Admin user is added without T_TYPE, which causes:
I added the Admin user with a T_TYPE of
L
based on what I could see from other spots in the code. It seems to work. Should line 1014 include a T_TYPE of L?The text was updated successfully, but these errors were encountered: