Fix/middleware

[github-actions[bot]]

Risk Level 2 - /home/runner/work/deep-license-plate-recognition/deep-license-plate-recognition/webhooks/middleware/protocols/openeye.py

The code is not handling the case where the 'plate' key might not exist in the nested dictionary, which could lead to a KeyError. Use .get() with a default value to avoid this issue.

---

Risk Level 2 - /home/runner/work/deep-license-plate-recognition/deep-license-plate-recognition/webhooks/middleware/protocols/soap.py

The code is not handling the case where the 'plate' key might not exist in the nested dictionary, which could lead to a KeyError. Use .get() with a default value to avoid this issue. Additionally, the presence of service_key in the code suggests that a sensitive key might be exposed. Ensure that no sensitive keys are hardcoded or logged.

---

Risk Level 3 - /home/runner/work/deep-license-plate-recognition/deep-license-plate-recognition/webhooks/middleware/protocols/nx.py

The process_request function is directly accessing nested keys which may not exist and could raise a KeyError. It's recommended to use .get() method with a default value for safer access. Additionally, the plate extraction logic is duplicated across multiple files; consider creating a utility function for reuse.

---

🔑🐍🔁

---

Powered by Code Review GPT

[AashishDhakal]

@ttayson Are you sure about the fields?

I see a different structure here:

[ttayson]

@AashishDhakal Visually it seems correct, but if you add the code to a JSON formatting tool, you can verify it. We may also need to update the guide with better formatting.

[marcbelmont]

@dibonjohnseron Please merge if you're also good with this.

[dibonjohnseron]

All good to me as well.