f2e-spec icon indicating copy to clipboard operation
f2e-spec copied to clipboard

Rework webserver

Open yoda-jm opened this issue 3 years ago • 8 comments

What does this PR do?

This enable webserver for all people using cpp-httplib that is easier to install (no dependency)

Additional Notes

There is still a fix to do in the CMake file in order to fetch it from a clone of cpp-httplib that we'll have on performous project.

Closes

Closes #603 Closes #452 Closes #623 Closes #517 (if https://github.com/performous/performous/pull/766#discussion_r875235203 is answered)

yoda-jm avatar May 17 '22 16:05 yoda-jm

I can't test until I get my laptop back. That being said, there are some things from my PR that I would very much like to see here, such as:

  • Refactoring in mime-type handling
  • Decoupling of the search between the webserver UI and the game's actual interface.
  • Ability to connect to webserver from local network only (i.e. define a subnet mask that allows more than localhost, but doesn't leave the computer open to the world)

There might be others, but that's what I remember now.

Also, I'm still not fully convinced that using a synchronous library will really work as expected.

Lord-Kamina avatar May 17 '22 17:05 Lord-Kamina

That being said, there are some things from my PR that I would very much like to see here, such as:

  • Refactoring in mime-type handling
  • Decoupling of the search between the webserver UI and the game's actual interface.
  • Ability to connect to webserver from local network only (i.e. define a subnet mask that allows more than localhost, but doesn't leave the computer open to the world)

I think it'd be wise to add this into separate pr's as this pr is currently only porting to another library

Baklap4 avatar May 17 '22 17:05 Baklap4

That being said, there are some things from my PR that I would very much like to see here, such as:

  • Refactoring in mime-type handling
  • Decoupling of the search between the webserver UI and the game's actual interface.
  • Ability to connect to webserver from local network only (i.e. define a subnet mask that allows more than localhost, but doesn't leave the computer open to the world)

I think it'd be wise to add this into separate pr's as this pr is currently only porting to another library

What about the subnet thing? I'm looking into whether httplib supports it at all but I can't find any info.

Lord-Kamina avatar May 17 '22 18:05 Lord-Kamina

Added both #603 & #452 as linked issues as these will be closed by merging this.

Baklap4 avatar May 17 '22 20:05 Baklap4

Converting this PR to draft status since it's outdated, got merge conflicts and has some open discussions.

Baklap4 avatar Sep 06 '22 17:09 Baklap4

@yoda-jm any objections that i take over this branch so i can make it up to date again?

Baklap4 avatar Sep 15 '22 17:09 Baklap4

@yoda-jm any objections that i take over this branch so i can make it up to date again?

Please go ahead :)

yoda-jm avatar Sep 15 '22 21:09 yoda-jm