Michael Cooper

Results 38 issues of 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...

Defect

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",...

Defect
P5 - Low Priority

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": {...

Enhancement

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...

Enhancement

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...

Enhancement

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,...

Defect
P1 - High Priority

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...

techdebt
P5 - Low Priority
task

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...

Enhancement
P5 - Low Priority

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...

P5 - Low Priority
improvement