Skip to content
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

minecraft-server 1.7.10+ & 1.8+ - EULA Acceptance Requirement Feature #1

Closed
jsgoecke opened this issue Sep 7, 2014 · 4 comments
Closed

Comments

@jsgoecke
Copy link

jsgoecke commented Sep 7, 2014

It appears Mojang is now requiring that one accepts the EULA before the server in the latest releases will run:

http://www.reddit.com/r/admincraft/comments/27dy0q/1710_forces_you_to_accept_the_eula_before_you_can/

I see that the latest run in this project is 1.7.9 just before this became a requirement. Is there a way to do this acceptance in the Dockerfile or way to resolve for the latest versions?

@itzg
Copy link
Owner

itzg commented Sep 7, 2014

This can definitely be solved, but I'm pondering how to do this in a way
that still honors the EULA. I'm thinking about requiring an env variable
passed on the command line and adjust the file when accepted. If not
provided I could have it output a message, offer the docker run info, and
exit.

What do you think?
On Sep 7, 2014 12:46 PM, "Jason Goecke" notifications@github.com wrote:

It appears Mojang is now requiring that one accepts the EULA before the
server in the latest releases will run:

http://www.reddit.com/r/admincraft/comments/27dy0q/1710_forces_you_to_accept_the_eula_before_you_can/

I see that the latest run in this project is 1.7.9 just before this became
a requirement. Is there a way to do this acceptance in the Dockerfile or
way to resolve for the latest versions?


Reply to this email directly or view it on GitHub
#1.

@itzg
Copy link
Owner

itzg commented Sep 7, 2014

I have pushed an update that includes the EULA acceptance on the command line along with the default being bumped up to 1.8.

@itzg
Copy link
Owner

itzg commented Sep 7, 2014

...forgot to mention that I updated the info with the new (required) option:
https://registry.hub.docker.com/u/itzg/minecraft-server/

@jsgoecke
Copy link
Author

jsgoecke commented Sep 8, 2014

Seems like a great approach. I will learn from this, as I am still coming up to speed on Docker, so that I may just submit a pull request next time.

@itzg itzg closed this as completed Sep 8, 2014
This was referenced May 20, 2020
@ghost ghost mentioned this issue Feb 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants