double render on legacy create

fablabbcn / smartcitizen-api

The Smart Citizen Engine

https://developer.smartcitizen.me

GNU Affero General Public License v3.0

10 stars 4 forks source link

Closed timcowlishaw closed 1 month ago

timcowlishaw commented 1 month ago

Fixes https://iaac.sentry.io/issues/5360077450/?project=5212681&query=is%3Aunresolved&referrer=issue-stream&statsPeriod=14d&stream_index=0, this was an error of mine. We shouldn't reuse the existing controller instance to render the json for forwarding, when we are within an http request.

sentry-io[bot] commented 1 month ago

This pull request was deployed and Sentry observed the following issues:

‼️ ArgumentError: argument out of range Sidekiq/HandleIncomingMQTTMessageJob View Issue
‼️ NoMethodError: undefined method `/' for "2024-05-22T06":String Sidekiq/HandleIncomingMQTTMessageJob View Issue
‼️ RuntimeError: Timestamp not within range Sidekiq/HandleIncomingMQTTMessageJob View Issue

_{Did you find this useful? React with a 👍 or 👎}