Open
Conversation
…o file delivery method respectively
Pull Request Test Coverage Report for Build 19363549474Details
💛 - Coveralls |
… to avoid lazy client from downloading whole BASE_DATA_DIR
…alsoDirs Tornado fetch file from central handler also dirs
…alsoDirs_debug Tornado fetch file from central handler also dirs debug
…delete add ability to delete files/dirs through API, but only in qiita test …
* general fixes 11-2025 * fix TestWorkflowHandler
* allow multiple standalone steps in workflows * flake8 * adding a test * 2 commands from start & extra commands * more tests * artifact.parents
* download public prep data * addressing @lucaspatel comments
* TST: study associations endpoint test * MAINT: be defensive on artifact prep assocation expectations * API: add /api/v1/study/<id>/associations to retrieve comprehensive id, path, processing information for a study * DOC: note schema, add some additiona doc strings and comments * LINT: pass w/ ruff check * LINT: now with flake8 * Rollback: allow an artifact to be unlinked * DOC: note internal rest endpoint * LINT: flake8 * Remove extraneous sort * An incomplete prep will not have an artifact
* add parents ArtifactHandler * fixing test * add job_id to ArtifactAPItestHandler * flake8
* flake8 to ruff * . import * check_analysis_access
* add analysis id to validate parameters * adding artifacts to job creation * add analysis to release * mv up pvals * artifacs -> artifacts * rm counts * rewrite scenarios * fix analysis error * no prep_template, parents or analysis provided * data_type * rm parents * params=None * overwrite_lock
* TST: study associations endpoint test * API: add /api/v1/study/<id>/associations to retrieve comprehensive id, path, processing information for a study * SQL: draft schema changes to support additional identifiers * API: unique_ids stub on base metadata object * TST: check unique_ids * TST: check unique_ids * API: passing SampleTemplate.unique_ids * API: passing PrepTemplate.unique_ids * API: passing Artifact.unique_ids * Constrain returned values * Based relative to Qiita schema * lint * Quality to schema
* 2026.01 * Apply suggestions from code review Co-authored-by: Amanda Birmingham <AmandaBirmingham@users.noreply.github.com> * rm å [skip ci] --------- Co-authored-by: Amanda Birmingham <AmandaBirmingham@users.noreply.github.com>
Corrected the spelling of 'January' and updated the description of new commands in 'qp-pacbio'.
…do_FetchFileFromCentralHandler
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
4 participants
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.
and either use nginx or tornado file delivery method respectively
I've also added the ability to fetch whole directories from Qiita main. As malicious clients could thus get the WHOLE content of base_data_dir, only directories can be fetched, which are a) listed as "directory" filetypes in Qiita's DB and b) where client provided a valid mountpoint
When Qiita is in testmode, files/dirs can now also be deleted trough an API endpoint, which is not exposed in productive mode.