zkbfetch Too many items (corporation_ids)

Please report bugs here, please be very detailed in your bug reports. If you've come up with some code to fix the bug, please post it here.
Post Reply
BluePeer
Noob
Posts: 3
Joined: Sat Jun 02, 2018 11:20

zkbfetch Too many items (corporation_ids)

Post by BluePeer » Sat Jun 02, 2018 11:22

ZKBApi: https://zkillboard.com/api/corporationID/98048488/
0 kills were posted and 7 were skipped (200 kills fetched). Timestamp of last kill: 2017-04-02 16:00:57
Error communicating with ESI, aborting!
[400] Error connecting to the API (https://esi.evetech.net/v2/corporations ... 2C98049848)
Feeds

{"error":"too many items for 'corporation_ids', 'corporation_ids' is required"}
remove the last entry (request 99 not 100) work
on esi site there write limit 100 oO

a switch of line
256 from class.helpers.php of common/includes/esi/
from
while($corporationIdsLength < 950 && count($corporationIds) > 0)
to
while($corporationIdsLength < 700 && count($corporationIds) > 0)
fixxed my current issue
but why charcounting ? there exist an entry limit
in my failer example the most id's 8 and some 9/10/11

i limit it now for me to 700 to avoid it for the future

User avatar
Salvoxia
Developer
Posts: 1528
Joined: Wed Feb 22, 2012 12:11

Re: zkbfetch Too many items (corporation_ids)

Post by Salvoxia » Sat Jun 02, 2018 19:24

Hi,

thank you for reporting.
This issue has already been fixed in the repository.

The character counting was implemented quite early in the ESI integration process, when the only concern was maximum URL length. In the fix, this has been changed to an item limit.
The first time this error was reported I looked for a limit in the ESI UI, but didn't find one. I looked again now, just to find the limit in the response, not the request specification. Maybe I missed it there before.

The fix will be included in the next release.

Best Regards,
Salvoxia

Post Reply

Return to “Bug Reporting”

Who is online

Users browsing this forum: No registered users and 23 guests