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

Why charactersheet on API? #94

Closed
FooDownunder opened this issue Feb 7, 2016 · 2 comments
Closed

Why charactersheet on API? #94

FooDownunder opened this issue Feb 7, 2016 · 2 comments
Assignees
Milestone

Comments

@FooDownunder
Copy link

One of the things that you mention is 'any security issues'.

Your api request states : Create a new key with an empty 'Access Mask (8).

Maybe you need some of this information, but you are asking for the full character sheet, which is a lot more than an 'empty' mask, including skills, implants and corporate roles. http://games.chruker.dk/eve_online/test_eve_api.php?function=character_charactersheet&myeve_key_id=

Looking at the source code, I can see calls to /account/APIKeyInfo.xml.aspx, /map/Jumps.xml.aspx, and /map/Kills.xml.aspx

These all will be provided with an empty access mask (0) and do not require character sheet.

Do you need the access mask 8 (character sheet) for pathfinder (or other) reasons? Is there a couple of typos on the api screen?

@exodus4d
Copy link
Owner

exodus4d commented Feb 7, 2016

Access mask (0) works of course. I don´t need any additional information. Registering with an empty mask should work fine. The (8) was just a relic from an older version. Ill fix the typo.

The support of CCPs´s XML APIs will be deprecated soon, once the CREST login is ready (#84). This will replace any user API Key. But for the next few weeks, we have to stick with those API Keys.

@exodus4d exodus4d self-assigned this Feb 7, 2016
@exodus4d exodus4d added this to the v 1.0.0RC3 milestone Feb 7, 2016
@exodus4d
Copy link
Owner

exodus4d commented Jun 3, 2016

Issue resolved in master branch

@exodus4d exodus4d closed this as completed Jun 3, 2016
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