Michael Cooper
Michael Cooper
https://sentry.prod.mozaws.net/operations/normandy-prod/issues/9938952/?referrer=github_plugin STR: Make a type=preferenceComparison filter object where instead of pref=foo.bar, you provide preference=foo.bar. ``` KeyError: 'pref' (15 additional frame(s) were not displayed) ... File "rest_framework/views.py", line 480, in raise_uncaught_exception...
This is caused by me changing the name of the filter object, and by me having created some testing recipes a while ago. We should handle this error somehow. Perhaps...
It looks like when we check certificates, we need to be a bit more careful about the exceptions that can be thrown by the network. https://sentry.prod.mozaws.net/operations/normandy-stage/issues/614319/ ``` ...snip File "normandy/health/api/views.py",...
Currently, we only expose recipe arguments and filter objects as a raw JSON string in graphql. For example, the query `{recipe(id:755){latestRevision{argumentsJson filterObjectsJson}}}` results in this response: ```json { "data": {...
The new contract tests work in CI (sometimes, see #1826), but don't pass against prod and stage. Looking at the problems in the test failures, it is definitely a shortcoming...
It would be useful to have a filter object that can filter by date. This is already possible in JEXL, and we've used it in practice. It can help future...
https://sentry.prod.mozaws.net/operations/normandy-stage/issues/5562863/ ``` KeyError: 'name' (17 additional frame(s) were not displayed) ... File "rest_framework/views.py", line 492, in dispatch response = handler(request, *args, **kwargs) File "rest_framework/mixins.py", line 84, in partial_update return self.update(request,...
In #1245, @peterbe is adding support for actual real OIDC authentication, where we handle the bearer tokens. After this has landed and we have removed all users of the old...
Recipe revisions should include a description field. This should be an optional field, and won't be included in the minimal recipe serializer used in `/api/v1/recipe/signed/`. This is different from #1082...
In #1165, @peterbe pointed out that it would be nice if invalid order-by keys would return a client error, instead of silently doing nothing. This sounds like a good idea...