Skip to content
This repository was archived by the owner on Jun 10, 2025. It is now read-only.

Conversation

@Ewarren7
Copy link

@Ewarren7 Ewarren7 commented Jul 3, 2018

This is to help investigate a long running problem where Bamboo is changing the employeeNumber they have for a User on their end (this corresponds to a person's Pistachio User id.

We are primarily interested in POST requests and for those we will also store the full log of the httparty debug of the request.

Merge Plan

  1. Merge this PR while Pistachio ENV BAMBOO_REQUEST_LOGGING_ENBALED is not set and the Gem file is not updated to ref this commit.

  2. Merge https://github.com/alphasights/pistachio/pull/9280 with migration for BambooRequestLog table and updated gem file to ref this

Note

I have no idea why tabular_data_update_row branch is used and not master in the Pistachio Gemfile

Copy link

@brianstorti brianstorti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤷‍♂️

@Ewarren7 Ewarren7 merged commit ccd3917 into tabular_data_update_row Jul 3, 2018
@Ewarren7 Ewarren7 deleted the eiw-return-http-request-info branch July 3, 2018 17:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants