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

Unexpected end of JSON input when using collection endpoint with large collection #18

Open
GamerKingFaiz opened this issue Jan 26, 2022 · 1 comment
Assignees
Labels
external something goes wrong, but depends on other library prio zero a prio zero bug to be solved

Comments

@GamerKingFaiz
Copy link

GamerKingFaiz commented Jan 26, 2022

I have made the following endpoint in my express server:

const getBggCollection = (req, res) => {
  const { username } = req.params;
  client.collection
    .query({
      username: username,
      own: 1,
      excludesubtype: "boardgameexpansion",
    })
    .then((result) => {
      console.log(result);
      res.send(result);
    });
};

I'm testing larger collections to makes sure this library handles them seamlessly. Unfortunately I've run into an issue where I get the following error when testing in Chrome.
image

In the screenshot above my test username is hermi and it just keeps erroring out.
I also tested Alan How and the first time I queried that username it gave the above error, but the second time it seemed to return normally (barring the error in #19).

I'm using the usernames collected in this BGG Thread for large collection testing.
Let me know if you need further information!

@LearningProcesss
Copy link
Owner

Problem come from Boardgamegeek itself
bggresp
Not sure, but i think that something is changing at their backend.

Catching this error have to be implemented.

@LearningProcesss LearningProcesss self-assigned this Jan 28, 2022
@LearningProcesss LearningProcesss added the external something goes wrong, but depends on other library label Jan 28, 2022
@LearningProcesss LearningProcesss added the prio zero a prio zero bug to be solved label Jan 29, 2022
@LearningProcesss LearningProcesss added this to the prio 0 - round one milestone Jan 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external something goes wrong, but depends on other library prio zero a prio zero bug to be solved
Projects
None yet
Development

No branches or pull requests

2 participants