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

Server agent #3342

Merged
merged 5 commits into from
Feb 5, 2015
Merged

Server agent #3342

merged 5 commits into from
Feb 5, 2015

Conversation

robn
Copy link
Member

@robn robn commented Feb 3, 2015

I've been talking about this for two years. Finally time to add it to master so I can think about building on top of it.

Basic usage is documented at http://pioneerwiki.com/wiki/ServerAgent

This is deliberately disabled to start with, to allow us to experiment. At some later time we'll work out the process for enabling it, where the server is, privacy concerns, etc. I don't want to this PR to get bogged down in those details.

It needs build stuff for MSVC. http://curl.haxx.se/download.html has libcurl binaries.

@robn robn changed the title Web agent Server agent Feb 3, 2015
The idea is simple - make a call to a remote method with some JSON data,
call a success function with some JSON data when its done, or call a
fail function.

This is deliberately not a generic HTTP client.

Two flavours here: the real thing that makes HTTP calls, and the Null
one which just does nothing
@fluffyfreak
Copy link
Contributor

That's cool, I guess I'll need to make sure we have a version of libcurl in thirdparty for VS2013 users.

Appropriate XKCD

@fluffyfreak
Copy link
Contributor

@robn web-agent branch has vs2013 fixes on it and PR for pioneerspacesim/pioneer-thirdparty#12

@robn
Copy link
Member Author

robn commented Feb 5, 2015

Thanks @fluffyfreak. I'll merge pioneer-thirdparty when this is merged.

And on that note, any particular objections to this? If not, I'll merge this evening!

@fluffyfreak
Copy link
Contributor

I might update the libcurl.dll but that can wait until it's merged.

I have no objections but I think it might need some documentation on what you can use it for :)

@robn
Copy link
Member Author

robn commented Feb 5, 2015

Well at this point nothing, because there's no server. Or from another angle, EVERYTHING, just a simple matter of programming :)

I have some ideas for a kind of player/community site with hooks into the game, but its nothing more than ideas and some paper notes. There's also things like querying for updates, sending in hardware info, that sort of thing. But obviously, needs server and client code.

Its a building block :)

@robn robn merged commit c89b3e6 into pioneerspacesim:master Feb 5, 2015
@robn robn removed the in progress label Feb 5, 2015
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

Successfully merging this pull request may close these issues.

2 participants