Skip to content

N°9338 - Add logging when endpoint returns HTTP error code.#65

Open
jbostoen wants to merge 2 commits intoCombodo:masterfrom
jbostoen:feature/log_http_status_errors
Open

N°9338 - Add logging when endpoint returns HTTP error code.#65
jbostoen wants to merge 2 commits intoCombodo:masterfrom
jbostoen:feature/log_http_status_errors

Conversation

@jbostoen
Copy link

Base information

Question Answer
Related to a SourceForge thread / Another PR / Combodo ticket? No
Type of change? Enhancement

Objective (enhancement)

Add logging about which HTTP status code is reported (in case of error).

It does not change any behavior.

Reproduction procedure (bug)

Have an external API that (occasionally) returns a HTTP error code.

Proposed solution

Just adding some logging. This helped me find out about an issue with a customer's iTop instance (a cache issue - resulting in a HTTP 500 error).

Checklist before requesting a review

  • I have performed a self-review of my code, and that it's compliant with Combodo's guidelines
  • I have tested all changes I made on an iTop instance
  • I have added a unit test, otherwise I have explained why I couldn't
  • I have made sure the PR is clear and detailled enough so anyone can understand the real purpose without digging in the code

@jf-cbd
Copy link
Member

jf-cbd commented Feb 27, 2026

Accepted in functional review

@jf-cbd jf-cbd moved this from First review needed to Pending functional review in Combodo PRs dashboard Feb 27, 2026
@jf-cbd jf-cbd moved this from Pending functional review to Pending technical review in Combodo PRs dashboard Feb 27, 2026
@jf-cbd jf-cbd changed the title Add logging when endpoint returns HTTP error code. N°9338 - Add logging when endpoint returns HTTP error code. Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pending technical review

Development

Successfully merging this pull request may close these issues.

4 participants