Project

Profile

Help

HostedRedmine.com has moved to the Planio platform. All logins and passwords remained the same. All users will be able to login and use Redmine just as before. "Read more...":https://support.plan.io/news/187

Feature #802523

closed

Refactor fcdb calls to be more flexible

Added by Zoltán Žarkov about 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Category:
Server
Sprint/Milestone:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

I want to change the login workflow to something that allows mechanisms other than unsalted md5 passwords. I propose changing user_load to user_exists and user_verify lua calls, where user_verify checks if the password is correct. To maintain compatibility, an md5sum function is exposed to the fcdb lua environment, and the default database.lua file calls it when checking password against database.


Files


Related issues

Related to Freeciv - Task #656466: S3_0 datafile format freeze (d3f)ClosedMarko Lindqvist2019-10-05

Actions
Actions #2

Updated by Zoltán Žarkov about 4 years ago

  • Related to Task #656466: S3_0 datafile format freeze (d3f) added
Actions #3

Updated by Marko Lindqvist about 4 years ago

Reviewing S3_0 patch: Documentation comment in script_fcdb.c shows that new user_save would now take just one parameter, but it still takes two parameters like before, right?

Actions #5

Updated by Marko Lindqvist about 4 years ago

  • Category set to Server
  • Status changed from New to Resolved
Actions #6

Updated by Marko Lindqvist about 4 years ago

  • Status changed from Resolved to In Progress

On S3_0 build without enabling auth:
src/server/scripting/script_fcdb.c:194:12: error: ‘md5sum’ defined but not used [-Werror=unused-function]
static int md5sum(lua_State *L)

This build has been configured as:
$ ./config.status --config
'--enable-debug' '--enable-client=gtk3.22' '--enable-fcmp=gtk3'

Actions #8

Updated by Marko Lindqvist about 4 years ago

  • Status changed from In Progress to Resolved
Actions #9

Updated by Marko Lindqvist about 4 years ago

  • Status changed from Resolved to Closed
  • Assignee set to Marko Lindqvist

Also available in: Atom PDF