Merged
Conversation
matthewpeterkort
added a commit
that referenced
this pull request
Dec 17, 2025
* Fix typo in logs (#137) * Update integration_tests.yaml (#139) * Update integration_tests.yaml * Update integration_tests.yaml * comment out logs to make it runnable within lfs custom transfer * pass back errors to git client * error logging for fence errors * Moving common function into correct module (#3) * update branch * logging for configure * fix logging so that errors are passed to the drs logger * fix parse config fun variable shadowing * change names * rename * fix deprecated ioutil, fix tests * feedbacks, fix tests * fix lint errors * fix lint * fix lint errors * Add functionality for token based authentication * delete and disable logs * Bugfix/remove transfer logs (#2) * revert something * another one * fix dir structure, make fence 503 resp also attempt to refresh the token * add single multipart * update funcs * cleanup progress bar to make it compat for git-drs * Add token checking to data-client * fix guid based upload via cleanup * cleanup logging * revert back to comment out logger * fix a bug where access token wouldn't get saved to cred leading to failed refresh token get request * patch an issue where full path was not getting used in upload * fix config bugs * bug fix * fix up data client to correctly update credentials on expired cred * Bugfix/multipart upload (#12) * try initial logging via `UploadSingleMultipartWithLogWriter` * try * patch progress bar io too * selectively disable progress bar but still log files * surface the reasons or whatever * improve logs call it rc3 * feature/monorepos (#11) * WIP: improve git-lfs compatability * no logs to stderr * fix up data client to correctly update credentials on expired cred * expand global config to accept auth token saving * Refactor Gen3Interface to embed credentials, simplifying API calls. Update multiple files to use the new interface methods, removing the need to pass profile configurations explicitly. This change enhances code readability and maintainability. * fix tests * Make logger support dependency injection * clean up ancinet deps, refactor. * cleanup logger * Leave no dep stone unturned * copilot feedbacks * adjust logger structs to be compatible with outside library * fix cosmetic issues --------- Co-authored-by: matthewpeterkort <matthewpeterkort@gmail.com> --------- Co-authored-by: quinnwai <quinnwai.wong@gmail.com> Co-authored-by: Brian <brian@bwalsh.com> * improve func * fix merge * Add changes from 'copilot/fix-multipart-upload-issues' branch to this refactored branch --------- Co-authored-by: Pauline Ribeyre <4224001+paulineribeyre@users.noreply.github.com> Co-authored-by: Krishna Agarwal <159047652+krishnaa05@users.noreply.github.com> Co-authored-by: quinnwai <quinnwai.wong@gmail.com> Co-authored-by: Kyle Ellrott <kellrott@gmail.com> Co-authored-by: Quinn Wai Wong <54592956+quinnwai@users.noreply.github.com> Co-authored-by: Brian <brian@bwalsh.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
need to do this so that the profileConfig can still be attached globally in jwt