Skip to content

feat: Support reading private key from string#13556

Merged
kodiakhq[bot] merged 8 commits intocloudquery:mainfrom
patriknordlen:github-app-credentials-from-env
Sep 1, 2023
Merged

feat: Support reading private key from string#13556
kodiakhq[bot] merged 8 commits intocloudquery:mainfrom
patriknordlen:github-app-credentials-from-env

Conversation

@patriknordlen
Copy link
Copy Markdown
Contributor

Summary

This PR addresses #13555 by adding a private_key configuration key that can be used to read the private key from a string instead of a file.

@patriknordlen patriknordlen requested review from a team and pilvikala and removed request for a team August 31, 2023 15:35
@patriknordlen patriknordlen changed the title feat: support reading private key from string feat: Support reading private key from string Aug 31, 2023
@patriknordlen patriknordlen changed the title feat: Support reading private key from string feat(github): Support reading private key from string Aug 31, 2023
@patriknordlen patriknordlen changed the title feat(github): Support reading private key from string feat: Support reading private key from string Aug 31, 2023
Copy link
Copy Markdown
Member

@erezrokah erezrokah left a comment

Choose a reason for hiding this comment

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

Thanks for following up @patriknordlen, added a couple of comments regarding the error messages

patriknordlen and others added 3 commits September 1, 2023 11:42
Co-authored-by: Erez Rokah <erezrokah@users.noreply.github.com>
Co-authored-by: Erez Rokah <erezrokah@users.noreply.github.com>
@patriknordlen
Copy link
Copy Markdown
Contributor Author

Thanks for following up @patriknordlen, added a couple of comments regarding the error messages

Thanks for the suggestions, accepted and committed!

@erezrokah erezrokah added the automerge Automatically merge once required checks pass label Sep 1, 2023
@kodiakhq kodiakhq bot merged commit 9909e24 into cloudquery:main Sep 1, 2023
kodiakhq bot pushed a commit that referenced this pull request Sep 1, 2023
🤖 I have created a release *beep* *boop*
---


## [7.3.0](plugins-source-github-v7.2.1...plugins-source-github-v7.3.0) (2023-09-01)


### Features

* Support reading private key from string ([#13556](#13556)) ([9909e24](9909e24))


### Bug Fixes

* **deps:** Update github.com/99designs/go-keychain digest to 9cf53c8 ([#13561](#13561)) ([a170256](a170256))
* **deps:** Update github.com/beatlabs/github-auth digest to 39adf75 ([#13563](#13563)) ([ea5f471](ea5f471))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.6.0 ([#13492](#13492)) ([c305876](c305876))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants