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

aql: add more examples, and add support to debug, such as printing built query string #131

Closed
LeiYangGH opened this issue Dec 11, 2019 · 2 comments · Fixed by #287
Closed
Assignees
Labels
good first issue Help Wanted We will be glad if somebody proposes a solution via PR

Comments

@LeiYangGH
Copy link

LeiYangGH commented Dec 11, 2019

when trying aql, with include, sort, and limit, I wasted a lot of efforts guessing what python equivalent syntax would be given the raw aql string.

I always got below error until I went over some issues and came up with the right solution:

requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: myhost/artifactory/api/search/aql

The error message is too general.

Could it be more acurate? such as some debugging functions to print the generated raw aql string?
And could any one add some example in readme? (if you allow, i'd like to add one example).

@allburov
Copy link
Member

May be enable debug logging for requests could help you for the moment? https://gist.github.com/Daenyth/b57f8522b388e66fcf3b

if you allow, i'd like to add one example

We have some examples in documentation with include, but it can don't cover enough https://devopshq.github.io/artifactory/#artifactory-query-language

Of course, feel free to contribute documentation and code!

@allburov allburov added good first issue Help Wanted We will be glad if somebody proposes a solution via PR labels Dec 11, 2019
@beliaev-maksim beliaev-maksim self-assigned this Sep 8, 2021
beliaev-maksim added a commit to beliaev-maksim/artifactory that referenced this issue Sep 8, 2021
@beliaev-maksim beliaev-maksim linked a pull request Sep 8, 2021 that will close this issue
beliaev-maksim added a commit to beliaev-maksim/artifactory that referenced this issue Sep 8, 2021
beliaev-maksim added a commit to beliaev-maksim/artifactory that referenced this issue Sep 13, 2021
@beliaev-maksim
Copy link
Member

now is added to develop branch

will be added in next release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Help Wanted We will be glad if somebody proposes a solution via PR
Development

Successfully merging a pull request may close this issue.

3 participants