8000 hardcoded localhost on query when create new user eprints 3.3 · Issue #518 · eprints/eprints · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

hardcoded localhost on query when create new user eprints 3.3 8000 #518

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

Open
kahidna opened this issue Nov 17, 2021 · 2 comments
Open

hardcoded localhost on query when create new user eprints 3.3 #518

kahidna opened this issue Nov 17, 2021 · 2 comments

Comments

@kahidna
Copy link
kahidna commented Nov 17, 2021

I try to create docker compose for eprints version 3.3 based on eprints 3.4 docker-compose from here.
but when creating sample repository from inside the container I got access denied when create database
image

after check the database, by default generated user's host is localhost. and I try to trace the query, and I found that localhost was hardcoded here.
image

and when I change the localhost to %, and then reconfig the database, everything work as expected.
image
but the user become redundant
image

and my question is,
is the hardcoded localhost is intentionally defined or for other reason?
and is there possibly issues when I change the localhost to % on the mysql.pm?

@drn05r
Copy link
Contributor
drn05r commented Nov 18, 2021

I was looking at the code recently and I was a bit puzzled why this was hardcoded. I suspect it has not caused anyone an issue up to now is that you would not be creating a user (or database) on a remote server, as they would require you to use another user account that has permissions to do this. Normally you would probably want to avoid having an account that can create other accounts from a remote server, so typically you would manually create the user and database on the remote server and then run "epadmin update" rather than "epadmin create". However, as that would mean having to manually run the other commands to initial setup EPrints (i.e. add an EPrints initial admin user, add subjects, generate static pages, etc.).

EPrints 3.3 is no longer being actively developed so I have created eprints/eprints3.4#182 where any change could be failry easily backported to 3.3

@drn05r
Copy link
Contributor
drn05r commented Nov 18, 2021

See eprints/eprints3.4@0e28f10 if you want to backport the change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants
0