Skip to content

authenticating with API key is broken; request overwrites X-Redmine-Api-Key header from BaseEngine.__init__ #353

Open

Description

@dmick

The fairly-new construct_request_kwargs() takes a param "headers" which it uses exclusively to create the request headers; however, if the engine object was instantiated with 'key', the key is stored in self.requests['headers'] already, and should be merged in construct_request_kwargs(), not overwritten

Activity

  1. dmick commented on Jul 25, 2025

    @dmick
    Author

    It looks like this was broken in #343

  2. erSitzt commented on Aug 13, 2025

    @erSitzt

    the maintainer of this project seems to be absent...

    #351

  3. dmick commented on Aug 13, 2025

    @dmick
    Author

    Thanks, yes, aware. Thought this might help, although I see a bug...new patch forthcoming

  4. dmick commented on Aug 13, 2025

    @dmick
    Author

    tweaked the patch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions