Skip to content

Add check for unbound user object before kafka producer step.#18

Merged
tjacovich merged 2 commits intoadsabs:mainfrom
tjacovich:kafka-producer-patch
Jun 12, 2025
Merged

Add check for unbound user object before kafka producer step.#18
tjacovich merged 2 commits intoadsabs:mainfrom
tjacovich:kafka-producer-patch

Conversation

@tjacovich
Copy link
Copy Markdown
Contributor

@tjacovich tjacovich commented Jun 11, 2025

When a user logs in, the current_user object becomes unbound from it's session, resulting in an error when the producer attempts to access the object for the purpose of generating the kafka message. This PR adds a check to make sure the user object is still bound.

@tjacovich tjacovich requested a review from femalves June 11, 2025 15:52
@coveralls
Copy link
Copy Markdown

coveralls commented Jun 11, 2025

Pull Request Test Coverage Report for Build 15613607092

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 5 (0.0%) changed or added relevant lines in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.2%) to 65.034%

Changes Missing Coverage Covered Lines Changed/Added Lines %
apigateway/services.py 0 5 0.0%
Files with Coverage Reduction New Missed Lines %
apigateway/services.py 1 75.86%
Totals Coverage Status
Change from base Build 15587748472: -0.2%
Covered Lines: 1142
Relevant Lines: 1756

💛 - Coveralls

@tjacovich tjacovich merged commit 2514713 into adsabs:main Jun 12, 2025
1 check passed
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.

3 participants