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

CSV Export not working #1119

Closed
drewguy opened this issue Nov 15, 2022 · 10 comments · Fixed by #1150
Closed

CSV Export not working #1119

drewguy opened this issue Nov 15, 2022 · 10 comments · Fixed by #1150

Comments

@drewguy
Copy link

drewguy commented Nov 15, 2022

CSV export from flights list page results in error reported by Cloudflare from openflights.

Works from settings page

@reedy
Copy link
Collaborator

reedy commented Dec 2, 2022

Mentioned in #1096 (comment)

@jpatokal
Copy link
Owner

jpatokal commented Dec 4, 2022

Unfortunately I'm unable to replicate, CSV exports from the flights page are working fine for me. I'm particularly mystified that it's working from Settings, since the data is obviously identical and they're calling the same backend to export.

@reedy
Copy link
Collaborator

reedy commented Dec 6, 2022

FWIW, neither work for me

https://openflights.org/php/flights.php?user=0&trid=0&alid=0&year=0&param=MAP&id=0&export=export

and

https://openflights.org/php/flights.php?export=backup

Both give "The web server reported a bad gateway error."

What do the error logs tell you? I'm getting it's a PHP fatal/similar...

@reedy
Copy link
Collaborator

reedy commented Dec 6, 2022

When I can get a local working dev install, I might be able to help a bit more ;)

reedy added a commit to reedy/openflights that referenced this issue Dec 23, 2022
@reedy
Copy link
Collaborator

reedy commented Dec 23, 2022

@drewguy Do you know how big your CSV backups have been previously?

@drewguy
Copy link
Author

drewguy commented Dec 27, 2022

Showing 457kb when I was able to do it using the other link . . . (I have about 500 flights with just flight data not extensive notes)

@drewguy
Copy link
Author

drewguy commented Dec 27, 2022

It's not getting the error now so I'll close . . . may have been temporary glitch

@drewguy drewguy closed this as completed Dec 27, 2022
@reedy
Copy link
Collaborator

reedy commented Dec 27, 2022

It's still broken for me - https://openflights.org/php/flights.php?export=backup

Mines only ~60KB with over 500 flights in it

reedy added a commit to reedy/openflights that referenced this issue Jan 2, 2023
@stillinbeta
Copy link

Broken for me as well, with 205 flights:

document.getElementById('apttable').select('tr').length
205 

@reedy
Copy link
Collaborator

reedy commented Jan 24, 2023

I've posted a fix that seems to work locally... Just needs review, merge, deployment and further testing :)

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 a pull request may close this issue.

4 participants