Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Set of patches for our instance #9

Closed
wants to merge 3 commits into from

Conversation

jdehaan
Copy link
Collaborator

@jdehaan jdehaan commented Feb 5, 2024

  • Squash commits for Net::LDAP instrumentation
  • Squash commits for GraphQL variables support
  • Squash commits for adding Peer.Service to Active_Job

After having a clean rebased PR we can keep that branch and make a tag used inside our software to reference to it.
On new upstream changes we can repeat the process and avoid having to deal with complicated merges

@jdehaan jdehaan requested a review from Hareramrai February 5, 2024 14:40
Hareramrai and others added 2 commits February 5, 2024 14:47
* fix: Track ldap status from Net::LDAP::PDU response

* fix: Add extra attribute for tracking false ldap error

* fix: OpenTelemetry error for NilClass

* fix: Ignore message & error_message if empty
* fix: Add graphql variables to the trace results

* fix: add tests
@jdehaan jdehaan force-pushed the chore/2024-02-05-rebase branch 2 times, most recently from 3fdac5a to 220bd7e Compare February 5, 2024 17:05
@jdehaan jdehaan force-pushed the chore/2024-02-05-rebase branch from 220bd7e to b6d0617 Compare February 5, 2024 17:11
@jdehaan jdehaan changed the title [WIP] chore: Rebase with all the custom patches cleanly applied as single commits feat: Set of patches for our instance Feb 5, 2024
@jdehaan
Copy link
Collaborator Author

jdehaan commented Feb 5, 2024

Created branch based on this PR: main-aoeur-20240205

@jdehaan jdehaan closed this Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants