macarthur-lab / matchbox

matchbox server for the Matchmaker Exchange
BSD 3-Clause "New" or "Revised" License
5 stars 2 forks source link

Email from Francois of a 400 error they are getting #9

Closed harindra-a closed 7 years ago

harindra-a commented 7 years ago

Email from Francois of a 400 error they are getting. This could be a JSON error they are getting, or something on our side. Looking into it.

Harindra

Getting an error back from Broad, namely 400 - "message not formatted properly and possibly missing header information” which I got yesterday too. I don’t have the JSON handy, I added logging to dump it next time I get the error. I’ll let you know when I get that, but wanted to alert you about this.

François

On Oct 4, 2016, at 8:20 AM, do-not-reply@phenodbmatcher.org wrote:

2016-10-04 08:20:01,977 ERROR match.py:4585 - 400 - Failed POST request, name: 'Broad', url: 'https://seqr.broadinstitute.org/api/matchmaker/v1', status: 400, request content type: 'application/vnd.ga4gh.matchmaker.v1.0+json; charset=utf-8', request accept: 'application/vnd.ga4gh.matchmaker.v1.0+json', response: '{"message":"message not formatted properly and possibly missing header information", "status":400}'

harindra-a commented 7 years ago

From Francois: looks like the error is on their side

Caught one, looks ok to me, except the institution & name but that is a user education issue at our end.

{ "_disclaimer": "People proposing to publish material that makes use of matches achieved through the GeneMatcher database agree to: Acknowledge GeneMatcher and the grant that supports it: 1U54HG006542; and Notify the systems administrator of a published paper (so that we can track our results), by sending an email through the 'Contact Us' tab on GeneMatcher. No one is authorized to attempt to identify patients whose results are in GeneMatcher by any means. This applies to matches made within GeneMatcher or with any other database included in the Matchmaker Exchange. GeneMatcher provides these data in good faith as a research tool, but without verifying the accuracy, clinical validity or utility of the data. GeneMatcher. Johns Hopkins University makes no warranty, express or implied, nor assumes any legal liability or responsibility for any purpose for which the data are used.", "patient": { "contact": { "href": "mailto:ssoden@cmh.edu", "institution": "None", "name": "None" }, "genomicFeatures": [ { "gene": { "id": "ENSG00000196767" } } ], "id": "9305", "inheritanceMode": "HP:0001419", "label": "POU3F4 alternate phenotype", "species": "NCBITaxon:9606"

harindra-a commented 7 years ago

Fixed, there was an indentation error on the slack notification function