Skip to content
Snippets Groups Projects

Pass controller runtime objects with Context

Merged Hossein Pursultani requested to merge hp-runtime-context into master
All threads resolved!

This MR introduces runtime context where runtime object of Controller, for example the Client, are embedded in the Context.

This make it easier to pass down the request-scoped information to other components.

The following are included:

  • ContextBuilder type for building a new runtime context with the Client, Logger, etc.
  • ClientFromContext function to retrieve the Client from a context.
  • RecorderFromContext function to retrieve the EventRecorder from a context.

Related to #889 (closed)

Closes #440 (closed)

Closes #325 (closed)

Edited by Hossein Pursultani

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Robert Marshall
  • Robert Marshall
  • Robert Marshall requested review from @mnielsen

    requested review from @mnielsen

  • added 1 commit

    • c6c224c0 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 1 commit

    • 1d820aad - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 1 commit

    • c6589b89 - Use optional arguments instead of Builder pattern

    Compare with previous version

  • Mitchell Nielsen
  • added 1 commit

    Compare with previous version

  • Mitchell Nielsen resolved all threads

    resolved all threads

  • mentioned in issue #325 (closed)

  • Thank you both :handshake:

  • Mitchell Nielsen approved this merge request

    approved this merge request

  • mentioned in commit 20e9dbe3

  • Mitchell Nielsen picked the changes into the branch 0-14-stable with commit 56da770c

    picked the changes into the branch 0-14-stable with commit 56da770c

  • mentioned in commit 56da770c

  • mentioned in issue #966 (closed)

  • Please register or sign in to reply
    Loading