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

Gh onprem connector #579

Merged
merged 4 commits into from
Nov 22, 2023
Merged

Gh onprem connector #579

merged 4 commits into from
Nov 22, 2023

Conversation

aperez-worklytics
Copy link
Contributor

@aperez-worklytics aperez-worklytics commented Nov 22, 2023

  • Adding support for customizing host on GH connector, useful for creating connections based on GitHubServer. If no setting is provided, it will use default endpoints for GitHub Cloud.
  • Fixed couple of broken links in docs.

Fixes

Features

Change implications

  • dependencies added/changed? no
  • something to note in CHANGELOG.md? no
    • anything that will show up in terraform plan/apply that isn't obviously a no-op? no
    • breaking changes? if in module/example that is NOT marked alpha, requires major version change no

@aperez-worklytics aperez-worklytics self-assigned this Nov 22, 2023
@aperez-worklytics aperez-worklytics marked this pull request as ready for review November 22, 2023 12:45
Copy link
Member

@eschultink eschultink left a comment

Choose a reason for hiding this comment

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

👍

don't worry about adding features like this to module-examples that don't cover BOTH google + msft; we'll get rid of those in the future.

@eschultink eschultink merged commit 49d9ca5 into rc-v0.4.41 Nov 22, 2023
@eschultink eschultink deleted the gh_onprem_connector branch November 22, 2023 15:45
eschultink added a commit that referenced this pull request Nov 23, 2023
* update release refs to rc-v0.4.41

* release tooling (#566)

* document release process

* imls for tools

* ensure example's main in good state before publish

* deterministic rule serialization (#567)

* deterministic serialization of Rules2

* add some comments on serialization

* more documentation (#565)

* improve main README

* provide some more security documentation

* Update docs/faq-security.md

Co-authored-by: aperez-worklytics <[email protected]>

* Update docs/aws/authentication-authorization.md

Co-authored-by: aperez-worklytics <[email protected]>

---------

Co-authored-by: aperez-worklytics <[email protected]>

* Always return scope from source, if missing in rules

* Remove IDE warning

* Remove IDE warning (put back Java8 compatiblity)

* Fix tests

* improve jira/zoom docs  (#569)

* reformat Jira cloud docs

* alignment

* more alignment

* add zoom screenshot

* zoom docs alignment

* jira note on needing a terminal

* test config takes precedence over rules (#571)

* fix java21 (#572)

* matrix build for java 21

* lombok upgraded to support jdk 21

* upgrade version numbers to fix refs to v21

* try upgrade junit, mockito

* fix build

* revert mocks to subclasses as needed

* try to fail back mockito behavior to keep java 11, 8 working

* standard build to java 21

* fix java version check

* upgrade apache commons lang3

* update workflows

* update README

* stronger java check, doc prereqs

* strengthen prereq check

* comment out 20, as can't fix it

* stop caching mvn packages and see if works

* fix indent

* get rid of skipping processing of dependencies

* add back clearing of mvn repo of our own artifacts, in case a problem

* stop tests of 18,19,20

* recover more sensible workflows

* fill h_4 pseudonym (v0.4 hash) in legacy case (#570)

* fill h_4 with default pseudonyms

* wip of fixing tests, migrating examples to non-legacy pseudonyms

* revert dbx example to original

* fix lots of examples

* fix dbx example

* fix more test cases

* fix more test cases

* fix rest of test cases

* fix test case

* rules cleanup (#575)

* more comments on Endpoint

* connector doc improvements

* improve health check (#576)

* add PSEUDONYMIZE_APP_IDS value to health check, if set

* simplify version handling in health checks, improve tests

* health check logs less verbose, as otherwise confuses people

* unused variable in tests

* revert version target

* remove multi-line strings, to preserve java 11 source compatibility atm

* fix test case

* switch-up mockito version, try to downgrade in java8 case

* disable java8 build

* try to fix java tests across platforms

* allow manual trigger of java8 workflow

* Fixed unit tests. (#577)

Co-authored-by: Erik Schultink <[email protected]>

* Gh onprem connector (#579)

* Allow customize github api host for supporting on prem versions

* Updated readme

* Fix broken link

* Fix broken link

* logic around optionals, to avoid creating hard dep on SOURCE

* avoid ever having 'empty' scope, which is non-sensical; want null or explicit value

---------

Co-authored-by: Erik Schultink <[email protected]>
Co-authored-by: Erik Schultink <[email protected]>
Co-authored-by: aperez-worklytics <[email protected]>
Co-authored-by: Valentyn Ponomarenko <[email protected]>
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