Draft: Wget simple, bearer token and Globus script API integrations #456
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
…(if they aren't found), rather than redirect the page.
* changes to fix wget download issues: multiple download, direct link handed to browser * Removed uneccessary import Co-authored-by: Sasha Ames <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* hotfix to get wget script download to work (#449) (#450) * hotfix to get wget script download to work (#449) * hotfix to get wget script download to work * fix removed return * Updating file to fix minor blank line issue. Co-authored-by: Sasha Ames <[email protected]> * Update typescript-eslint monorepo to v5.30.7 Co-authored-by: Carlos Downie <[email protected]> Co-authored-by: Sasha Ames <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Bumps [terser](https://github.com/terser/terser) from 5.12.1 to 5.14.2. - [Release notes](https://github.com/terser/terser/releases) - [Changelog](https://github.com/terser/terser/blob/master/CHANGELOG.md) - [Commits](https://github.com/terser/terser/commits) --- updated-dependencies: - dependency-name: terser dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [moment](https://github.com/moment/moment) from 2.29.2 to 2.29.4. - [Release notes](https://github.com/moment/moment/releases) - [Changelog](https://github.com/moment/moment/blob/develop/CHANGELOG.md) - [Commits](moment/moment@2.29.2...2.29.4) --- updated-dependencies: - dependency-name: moment dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Carlos Downie <[email protected]>
…into wget-simple
Codecov Report
@@ Coverage Diff @@
## v1.0.9 #456 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 78 62 -16
Lines 2155 1739 -416
Branches 330 326 -4
==========================================
- Hits 2155 1739 -416
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 42 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
* Testing the use of a central config file and a manager script. Created a metagrid_configs folder where different configs can be stored and used. * updates to scripts and dockerfile for traefik to handle empty prefx variables. * More fixes and updates. * Refactoring to fix issues with the variables used in the master config file, so that they don't repeat in the config. Removed unnecessary folders and updated the manage_metagrid script to allow saving and restoring of config files in an automated way. Every time changes are made to the config, a backup will be created which will have date and time associated to it. Users can then restore that backup with a simple command in the script (which will also save the current config that is replaced. Note that if lots of changes are made, config backups should probably be deleted now and then. * Updated script to include sudo that was missing and also copy to traefik when a backup is restored. * updates t the script for local testing. Fixed a docker-compose minor issue in the backend production so it points to correct config file. * Updated the scripts to save environment file in each docker directory. It seems that the frontend uses the package.json build call rather than the one in docker compose, so the environment file was not found when building react. Modifications should help fix that issue. * Added cors proxy variable because it still seems necessary for the backend communication with frontend... * Updates to the metagrid documentation. A restructuring update needs to be done for getting_started_production.md in order to reflect the new way which configuration is setup and updated in production. * Updated manage script to include the option for starting/stopping the docs local containers. Updated django version to 3.2.15 * Updated the documentation to specify the manage_metagrid.sh scripts and how to update the projects. Also updated the configuration steps and how to get production started.
…ed some mocks for the temp storage calls which were breaking several tests. Created a new test file for the DatasetDownload component, however it needs to have tests added. Next step is to examine coverage and also fix the skipped tests if neede to improve coverage.
… the download has finished.
…ed keycloak related changes for now. Fixed tests that were broken from the wget changes, converted to using post and tests passed again.
…the news section.
sashakames
changed the title
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.
Description
Integrates the frontend with the updated Wget API. Following changes:
Fixes 316
Type of change
How Has This Been Tested?
Checklist