-
Notifications
You must be signed in to change notification settings - Fork 6
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
Merging Main to Plugin #869
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Making more util trcsub for common. * Apply state changes. * Checking in progress. * Reverting some hardcodings. * Adding some robustness handling and changes to initial states. * More state updates for windows side. * Allow for definition of service. * Adding Refresh syncMode (#775) * Adding refresh * Adding refresh logs --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com>
* Fix missing import. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com>
* Fix broken build (#778) * Fix missing import. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com> Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: meghan-bailey <107890930+meghan-bailey@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Better support for server cert. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com>
* Update to go 1.21.4 * More robust state handling. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com>
* Prevent premature exits. * Switch to ranging over channel. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com>
* Fixing go.mod --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
* Cleanup config and certify to handle errors more gracefully for better interaction with trcsh. Support multiple deployment configurations on a box. * Certify exit with error code. * Fixing some crashing. * Let this happen async. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
* Add some error handling with early exit. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
* Make deployments re-runnable. * Set timer for each run. Make CtlMessage a buffered channel. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
* Cleanup config and certify to handle errors more gracefully for better interaction with trcsh. Support multiple deployment configurations on a box. * Certify exit with error code. * Fixing some crashing. * Let this happen async. * Adding some more commands. * Select more suitable param name. * Add some error handling with early exit. * Make deployments re-runnable. * Set timer for each run. Make CtlMessage a buffered channel. * Couple more fixes for multi-deploy. * Small spelling correction. * Get everything at least running... Output to correct directory not yet functional. * A little more cleanup. * Add support for projectservice. * Add read support for projectservice. * Fix missing import. * Fixing couple bugs and log cleanup. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
* Revert breaking change. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
* Prepare for release. (#852) * Some mod cleanup. * Updating to go 1.21.5 --------- * Merging Develop into Main (#854) * Merging Develop into Main (#855) --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
* Adding support in trcshell for environments like dev-1. * Provide env overrides. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
* Adding support in trcshell for environments like dev-1. * Provide env overrides. * Support plugin config loading even when utilizing custom environments. ---------
* Fix locking issue. * Roll back since not needed anymore. * Update to more concurrent safe go-mysql * Update to latest go-mysql-server build. --------- Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
* Adding bounds check. * Add actual bounds checking. * Bounds check totalKeyShard * Disallow complex path navigation. * Add log sanitizing. * More sanitization. * Directly applying recommendation from dependabot ---------
joel-rieke
approved these changes
Dec 14, 2023
joel-rieke
added a commit
that referenced
this pull request
Dec 14, 2023
* Various codeql fixes. (#865) * Adding bounds check. * Add actual bounds checking. * Bounds check totalKeyShard * Disallow complex path navigation. * Add log sanitizing. * More sanitization. * Directly applying recommendation from dependabot --------- Signed-off-by: dependabot[bot] <support@github.com> * Merging Develop into Main (#868) * Merging Main to Plugin (#869) * Adding addr to modifer cache key --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
joel-rieke
pushed a commit
that referenced
this pull request
Dec 20, 2023
joel-rieke
added a commit
that referenced
this pull request
Dec 20, 2023
* Various codeql fixes. (#865) * Adding bounds check. * Add actual bounds checking. * Bounds check totalKeyShard * Disallow complex path navigation. * Add log sanitizing. * More sanitization. * Directly applying recommendation from dependabot --------- Signed-off-by: dependabot[bot] <support@github.com> * Merging Develop into Main (#868) * Merging Main to Plugin (#869) * Adding addr to modifer cache key --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
joel-rieke
pushed a commit
that referenced
this pull request
Dec 20, 2023
joel-rieke
added a commit
that referenced
this pull request
Dec 20, 2023
* Various codeql fixes. (#865) * Adding bounds check. * Add actual bounds checking. * Bounds check totalKeyShard * Disallow complex path navigation. * Add log sanitizing. * More sanitization. * Directly applying recommendation from dependabot --------- Signed-off-by: dependabot[bot] <support@github.com> * Merging Develop into Main (#868) * Merging Main to Plugin (#869) * Adding addr to modifer cache key --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
joel-rieke
pushed a commit
that referenced
this pull request
Dec 20, 2023
joel-rieke
added a commit
that referenced
this pull request
Dec 20, 2023
* Various codeql fixes. (#865) * Adding bounds check. * Add actual bounds checking. * Bounds check totalKeyShard * Disallow complex path navigation. * Add log sanitizing. * More sanitization. * Directly applying recommendation from dependabot --------- Signed-off-by: dependabot[bot] <support@github.com> * Merging Develop into Main (#868) * Merging Main to Plugin (#869) * Adding addr to modifer cache key --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
joel-rieke
pushed a commit
that referenced
this pull request
Jan 10, 2024
joel-rieke
added a commit
that referenced
this pull request
Jan 10, 2024
* Various codeql fixes. (#865) * Adding bounds check. * Add actual bounds checking. * Bounds check totalKeyShard * Disallow complex path navigation. * Add log sanitizing. * More sanitization. * Directly applying recommendation from dependabot --------- Signed-off-by: dependabot[bot] <support@github.com> * Merging Develop into Main (#868) * Merging Main to Plugin (#869) * Adding addr to modifer cache key --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Karnveer Gill <101755160+KarnveerSGill@users.noreply.github.com> Co-authored-by: Karnveer Gill <karnveer_gill@trimble.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.