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

PHP message: Validation failed: "name"" while reading response header from upstream #147

Closed
Tupsi opened this issue May 13, 2016 · 7 comments
Assignees
Milestone

Comments

@Tupsi
Copy link
Contributor

Tupsi commented May 13, 2016

Currently trying out the development branch. It seems that every (insert poor or stupid here) user who had the idea to use a ' in his eve character name can not login via SSO. I guess the character name needs to be properly escaped somewhere?

Full message reads:

2016/05/13 13:27:00 [error] 11392#11392: *6681 FastCGI sent in stderr: "PHP message: Validation failed: "name"" while reading response header from upstream, client: xxx.xxx.xxx.xxx, serve
r: our.thing.net, request: "GET /sso/callbackAuthorization?code=blablacode HTTP/2.0", upstrea
m: "fastcgi://whereIputit.com", host: "our.thing.net", referrer: "https://login.eveonline.com/oauth/authorize?response_type=code&redirect_uri=htps..."

@exodus4d
Copy link
Owner

Very interesting! I spend some time searching for an explanation for this. I figured out multiple possible code blocks where where this could happen.

Could you please provide a more detailed error log (set DEBUG = 3 in app/config.ini). So we have a backtrace to worke with.

exodus4d added a commit that referenced this issue May 14, 2016
@Tupsi
Copy link
Contributor Author

Tupsi commented May 15, 2016

where would I find that extra debug info? The error.log seems as before, same with the nginx log.

@Tupsi
Copy link
Contributor Author

Tupsi commented May 15, 2016

forget my last remark, for whatever reason the debugging is on now and there are more strange new lines in the nginx error log. Trying to get a hold on the person with he ' in the character names now to give you anything usefull.

@Tupsi
Copy link
Contributor Author

Tupsi commented May 15, 2016

btw, is there anyway we can have a chat instead of this constant github mailing in between? Your hanging out on any irc channels or something more modern?

@exodus4d
Copy link
Owner

@Tupsi yeah sure. Send me an Email to [email protected] . Ill invite you to Slack:

https://pathfinder-eve-online.slack.com

exodus4d added a commit that referenced this issue May 15, 2016
- #147 improved CREST API error logging (WIP)
- improved SSO controller (removed access_token from public endpoints)
@exodus4d
Copy link
Owner

Found the error: It has nothing to do with " ' " in the character name. It is the length of the name. The character name length must be <= 25 characters. Ill remove/adjust this validation rule and pull my changed when Im back home.

@exodus4d exodus4d added this to the v 1.0.0RC3 milestone May 17, 2016
@exodus4d exodus4d self-assigned this May 17, 2016
exodus4d added a commit that referenced this issue May 17, 2016
@exodus4d
Copy link
Owner

User names can now be up to 50 characters long

exodus4d added a commit that referenced this issue Jun 3, 2016
* #84 test data dump from CREST login

* updated "credits" dialog (Google+ link)
fixed login form layout

* updated Cortex Data-Mapper

* - #84 CREST Login (WIP)
- New CREST controller
- Database restructuring
- improved type-casting for some controller functions
- New login process
- Fixed some bugs during the setup process (/setup root)
- Added CREST request caching by response headers

* pathfinder-84 [Feature Request] CREST Pilot Tracking, many smaller Bugfixes

* pathfinder-84 [Feature Request] added develop JS files

* closed #121 fixed wormhole signature type caching

* closed #120 removed map-loading animation for larger maps (same behaviour as IGB)

* closed #119 fixed wormhole signature id count

* closed #114 Added check for already existing system when adding a new one. (fixed PDO 'duplicate entry' error)

* closed #112 fixed DataTables error for missing "status" data (signature table)

* closed #111 fixed convertDataToUTC(); client side date transformation

* closed #109 fixed system TrueSec rounding

* closed #103 fixed system updated timestamp in getData()

* fixed CSS class for secStatus in Routes module

* closed #121 fixed wormhole signature type caching

* changed dateTime format from German to US format
fixed some minor bugs in signatureTable module

* closed #81 fixed "signature type" overwriting by "signature reader" update

* closed #106 added new signature_types form C5/6 wormholes (gas/ore)

* closed #129 fixed parameter hinting

* closed #131 new "route search" algorithm, added current map systems to live search, added refresh/update functionality for each found route, added bulk route refresh function, added "meta map" route search (search on multiple maps), added route "filters" (restrict search on "stargates", "wormholes", "jumpbridges"), added route "filter" for wormholes (reduced/critical wormholes)
closed #89 fixed "loop connections" on same system
#84 added error messages for "invalid" CREST "Client ID"
added "bootboxjs" (customized styled checkboxes/radio buttons) CSS only
"Font Awesome" version upgrade 4.4.0 -> 4.61
"Bootbox.js" version upgrade 4.3.0 -> 4.4.0
fixed "system dialog" (added responsive layout)

* closed #134  fixed db column type DT_INT (8 bytes) to DT_BIGINT

* closed #138 added new cookie based login

* closed #137 fixed javascript errors on trying to establish an "invalid" connection

* - #84, #138 improved "character selection" on login page (expired cookies are deleted, character panel layout improvements)
- added new "Server info panel" to the login page
- added new cronjob to delete expired cookie authentication data

* #138 enables character switching between characters which have same user

* - PHP Framework upgrade 3.5.0 -> 3.5.1 (fixes some issues with CREST cURL caching, and SESSION management)
- #138 added "cookie logout" to "logout" menu entry

* - updated "feature page" with new feature descriptions and label
- added some new images to the "feature gallery"
- removed "beta" status from "magnetizing" feature on map menu
- hide "server status" panel on "mobile" breakpoint

* - #138 clear character authentication data on sold characters

* closed #142 added custom "onsuspect()" session handler

* #142 do not log suspect if no file is defined in pathfinder.ini

* #142 added NullSec Data/Relic sites to C1/2/3 wormholes as signature option

* #144 fixed "Character not found" warning

* #144 fixed "Character not found" warning

* closed #144 fixed broken routes panel in IGB

* updated README.md for upcoming release

* #147 response header validation

* #149 changed comment for 'BASE' framework var

* fixed map  import

* - added minimal SDE dump (EVE Online: Citadel)
- #147 improved CREST API error logging (WIP)
- improved SSO controller (removed access_token from public endpoints)

* closed #154 added alliance maps to CREST API

* - updated Gulp build dependencies
- increased CREST timeout from 3s -> 4s
- added "Accept" Headers for some CREST endpoints

* cloased #147

* - closed #153 added character verification check for getAll(); Signatures Ajax endpoint

* - updated README.md (added Slack developer chat information)

* Bugfix frig holes (#159)

* added missing frigate wormholes and fixed Q003 destination in shattered wormholes

* changed C7 to 0.0 for Q003

* - fixed broken "graph" data for system

* added a  "failover" system  for bad crest requests (HTTP status 5xx,.. )

* Red Gaint => Red Giant (#161)

* closed #163 added CREST endpoint support for "waypoints"

* fixed typo

* closed #160 fixed tooltip container

* - added new features to login page

* closes #154 added alliance map support

* fixed XML path for cronjobs

* fixed a bug with inactive "private" maps

* closes #175 added alternative environment configuration

* - v1.0.0  build
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