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

have send2cgeo work on project-gc.com #2

Open
Lineflyer opened this issue Jun 26, 2015 · 4 comments
Open

have send2cgeo work on project-gc.com #2

Lineflyer opened this issue Jun 26, 2015 · 4 comments

Comments

@Lineflyer
Copy link
Member

From @simrebel on June 17, 2015 19:47

I would like to have send2cgeo to work on project-gc.com

Copied from original issue: cgeo/cgeo#5040

@Lineflyer
Copy link
Member Author

AFAIR there was some discussion about this with @ganja1447 some months ago and AFAIR he already prepared at least a menu item in the V-GPS for it for later use.

@magma1447
Copy link

There are two problems with send2cgeo to solve for this to work.

We have to be able to do POSTs instead of GETs. Due to the (not specified) size limit of GET requests.

CORS (Cross-origin resource sharing) must be enabled in send2cgeo.
The problem that CORS solves is that web clients by default don't allow to use JS on other domains. But the client can ask the web server if it's ok. This snippet will tell the client that it's ok, and should be inserted before any other output. http://pastebin.com/4hwjP8NZ
We could probably shorten the allow-methods list to POST only.

@mirabilos
Copy link

GET requests are specified for up to 1024 bytes, although most servers take longer ones (usually twice, four, or eight times the size, sometimes even 16 KiB). However, for such sizes, POST is really more appropriate… actually, POST is always the appropriate one if state is changed on the server.

Is there anything I (experienced programmer) can do to help?

@mirabilos
Copy link

ganja from pgc said: “As I understand it, their server side code isn't available on Github, which makes it hard to help out.”

Lineflyer added a commit that referenced this issue Jan 8, 2020
Lineflyer pushed a commit that referenced this issue Aug 19, 2020
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

4 participants