-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
Updating readme with more detail #13
base: master
Are you sure you want to change the base?
Conversation
README.md
Outdated
@@ -8,19 +8,50 @@ | |||
|
|||
# Installation | |||
|
|||
1. `npm install screepsmod-auth` in your server folder. | |||
2. Thats it! | |||
1. Either run `npm install screepsmod-auth` in your screeps private server folder or subscribe the Steam workshop https://steamcommunity.com/sharedfiles/filedetails/?id=800390576 if your using the in-game private server. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably be split into two subsections, npm version
and steam version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
README.md
Outdated
|
||
# Usage | ||
2. Modify mods.json and add a new line in your mods list pointing to this mod like so: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only applies to npm version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated
README.md
Outdated
], | ||
``` | ||
3. Relaunch your screeps private server. | ||
3. Follow the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicate number
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
``` | ||
setPassword('yourUserName', 'yourPassword') | ||
``` | ||
Note: that setPassword says to pass in the email. Don't do this. Instead give it your username. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If an email is set during step 1, it will accept either email or username
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just checked the code again, as I could have sworn I updated the README, trying setPassword without the correct params does actually say use username, and the code doesn't accept email for this one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright, I'll leave it as it is as I think your saying my readme edits are correct right?
README.md
Outdated
2. Goto http://yourServerHostOrIP:21025/authmod/password/ | ||
1. Connect to your screeps private server once to create your account. You don't need to set an email as that won't be used. Just username and password is important. | ||
|
||
2. Send a POST call to http://yourServerHostOrIP:21025/authmod/password/ with this header: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will not work, it requires a valid login token in order to set the password. The web form takes care of that by having the steam login button to obtain the token.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So does the web form method only support steam then? I'm using the npm method and I had to use fiddler, edit the POST call according and reply it, and it returned 200. Not sure if it actually worked however
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It support both steam and npm, it uses the steam oauth system to get a valid steam auth, which the server then returns a server auth token to the form. It uses that token for authentication to set the password. it needs to authenticate the user first in order to prevent just anyone from setting the password. Since it uses the steam login system, it also auto-creates the user if it doesn't exist. However, I have been seeing it take two tries to set the password. I haven't had the time to poke at that and figure out the cause yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
take a look again
|
||
## Web Form Method | ||
1. Connect to your screeps private server once to create your account. | ||
2. Send a POST call to http://yourServerHostOrIP:21025/authmod/password/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is still not valid, as mentioned in the previous comments, this will not work. Needs both the steam auth and the password data or it will not apply.
No description provided.