Fix "missing datasource" error

grafana / sqlds

A package that assists writing SQL-driven datasources

Apache License 2.0

20 stars 12 forks source link

Fix "missing datasource" error #51

Closed andresmgot closed 3 years ago

andresmgot commented 3 years ago

The datasourceId field in the JSON body of the query is deprecated and not meant to be used. This PR:

Uses the datasource UID rather than the ID
Gets the UID from the request context, not the query body

It fixes a temporary issue with Grafana 8.3-pre.

ryantxu commented 3 years ago

With https://github.com/grafana/grafana/pull/41769 this should not be required -- however, it is better that we stop using datasourceId directly so that future changes are less painful