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

feat: add ? to query #159

Merged
merged 3 commits into from
Jun 1, 2022
Merged

feat: add ? to query #159

merged 3 commits into from
Jun 1, 2022

Conversation

sashamelentyev
Copy link
Contributor

Наткнулся на кейс, если в ямле в query-параметрах явно не стоит ? - то вылетала 404. Есть предположение, что адрес конкатенировался как path+query.
Пример:

- name: name
  method: GET
  path: /api/v1/collections
  query: limit=10&offset=0

@leorush
Copy link
Contributor

leorush commented May 30, 2022

Кажется, ещё неплохо бы дать возможность отключать такое поведение. Вдруг у кого-то не стандатная схема uri используется? А мы ему вопрос добавим.

@sashamelentyev
Copy link
Contributor Author

@leorush query же и подразумевает query-параметры, в рамках uri знак вопроса и нужен чтоб их передать.
По конфигурации в поле query знак вопроса выглядит избыточным как по-мне. С таким успехом можно и в конце path ставить знак вопроса, и это кстати сработает:

- name: name
  method: GET
  path: /api/v1/collections?
  query: limit=10&offset=0

@leorush
Copy link
Contributor

leorush commented May 30, 2022

Не спорю, я к тому что может быть кто-то использует нестандартный uri, написал тесты, а мы ему breaking change.

@sashamelentyev
Copy link
Contributor Author

По-идее, можно в релизе это указать

@Cdayz
Copy link
Contributor

Cdayz commented May 31, 2022

@sashamelentyev Добавь пожалуйста информацию о изменениях в README

@sashamelentyev
Copy link
Contributor Author

@Cdayz, добавил

@Cdayz Cdayz added enhancement New feature or request patch release as patch labels Jun 1, 2022
@Cdayz Cdayz merged commit 14a6a25 into lamoda:master Jun 1, 2022
@github-actions
Copy link

github-actions bot commented Jun 1, 2022

🚀 PR was released in v1.16.1 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request patch release as patch released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants