[Bug] JSON error message when project has no readme

[bishabosha]

Current behavior

https://index.scala-lang.org/ruimo/csvparser

Indeed the project as of writing has no readme on github

Expected Behavior

Perhaps fill the panel with "No README found for this project, please check the repository"

Extra comments

No response

Search terms

README, error message

[adpi2]

Thanks for the report. Would you like to submit a PR? :)

[mfirry]

Could this be solved by adding something like case HttpResponse(StatusCodes.NotFound, headers, entity, _) => in here?

[adpi2]

@mfirry Thanks for jumping into this. The issue is here: https://github.com/scalacenter/scaladex/blob/13ec905ae2b509b58e7c95235e17a5347552b312/modules/webclient/src/main/scala/scaladex/client/Client.scala#L43-L52

At the client side, we send a request to Github to get the readme but we don't check the status code.

[mfirry]

Ok so something along these lines would work for you?

    fetch(request).toFuture
      .flatMap { res =>
        if(res.status == 200) {
          res.text().toFuture
        } else {
          Future("No README found for this project, please check the repository")
        }

[adpi2]

Ok so something along these lines would work for you?

Yep it looks good