Loïc Lecrenier
56e79fa850
Update task snapshot test and clean up details
2023-01-17 13:19:04 +01:00
Loïc Lecrenier
c71a8ea183
Update to latest milli and deserr
2023-01-17 13:10:38 +01:00
Loïc Lecrenier
b781f9a0f9
cargo fmt
2023-01-17 11:07:07 +01:00
Loïc Lecrenier
07b90dec08
Remove unused proptest dependency
2023-01-17 11:07:07 +01:00
Loïc Lecrenier
9194508a0f
Refactor query parameter deserialisation logic
2023-01-17 11:07:07 +01:00
Loïc Lecrenier
49ddaaef49
Fix missing_swap_indexes error code and handling of expires_at param...
...
of create api key route
2023-01-17 09:43:07 +01:00
Loïc Lecrenier
766dd830ae
Update deserr to latest version + add new error codes for missing fields
...
- missing_api_key_indexes
- missing_api_key_actions
- missing_api_key_expires_at
- missing_swap_indexes_indexes
2023-01-17 09:43:07 +01:00
Loïc Lecrenier
436ae4e466
Improve error messages generated by deserr
...
Split Json and Query Parameter error types
2023-01-17 09:43:07 +01:00
bors[bot]
a5c4fbbcea
Merge #3334
...
3334: Add specific error codes `immutable_...` r=irevoire a=loiclec
Add the following error codes:
When an immutable field of API key is sent to the `PATCH /keys` route:
- `ImmutableApiKeyUid`
- `ImmutableApiKeyKey`
- `ImmutableApiKeyActions`
- `ImmutableApiKeyIndexes`
- `ImmutableApiKeyExpiresAt`
- `ImmutableApiKeyCreatedAt`
- `ImmutableApiKeyUpdatedAt`
When an immutable field of Index is sent to the `PATCH /indexes/{uid}` route:
- `ImmutableIndexUid`
- `ImmutableIndexCreatedAt`
- `ImmutableIndexUpdatedAt`
Co-authored-by: Loïc Lecrenier <loic.lecrenier@me.com>
Co-authored-by: Tamo <tamo@meilisearch.com>
2023-01-12 15:31:38 +00:00
Tamo
21b8cd53b7
reformat
2023-01-12 16:20:24 +01:00
Loïc Lecrenier
7f80b116bc
Add specific immutable_field error codes
2023-01-12 16:20:14 +01:00
bors[bot]
341f8478b4
Merge #3330
...
3330: test the error codes on the task routes + fix the missing error codes on the limit and from r=dureuill a=irevoire
Co-authored-by: Tamo <tamo@meilisearch.com>
Co-authored-by: Loïc Lecrenier <loic.lecrenier@me.com>
2023-01-12 15:02:44 +00:00
Tamo
79c7f65c30
make a test more reliable
2023-01-12 15:39:28 +01:00
bors[bot]
2bc60c29fc
Merge #3336
...
3336: Add missing `needs:` to the git latest tag workflow r=curquiza a=curquiza
Fixes this problem: the workflow to update the latest git tag was triggered despite the first check failed
<img width="580" alt="Capture d’écran 2023-01-12 à 15 07 00" src="https://user-images.githubusercontent.com/20380692/212087926-975eb387-c8c9-4789-8a62-a56143b9bbd4.png ">
These leads to update our latest git tag: our latest git tag corresponds to the `v1.0.0-rc.0` tag instead of `v0.30.5`. (I'm fixing this right now)
<img width="586" alt="Capture d’écran 2023-01-12 à 15 08 15" src="https://user-images.githubusercontent.com/20380692/212088136-f4bc2e9c-d824-4c23-8213-52598c742ebd.png ">
Co-authored-by: curquiza <clementine@meilisearch.com>
2023-01-12 14:24:31 +00:00
curquiza
680ea39bba
Add missingneeds: to the git latest tag workflow
2023-01-12 15:04:11 +01:00
Tamo
a524dfb713
fix the analytics
2023-01-12 14:49:50 +01:00
Tamo
705fcaa3b8
reformat the imports
2023-01-12 14:09:15 +01:00
Loïc Lecrenier
a09b6a341d
Move tasks route to deserr
2023-01-12 13:57:29 +01:00
Tamo
387874ea26
test the error codes on the task routes
2023-01-12 13:46:19 +01:00
bors[bot]
5c1a7c3b9a
Merge #3329
...
3329: Refactor error handling from deserr r=irevoire a=loiclec
Close https://github.com/meilisearch/meilisearch/issues/3318
Close https://github.com/meilisearch/meilisearch/issues/3289
[TODO]
Co-authored-by: Loïc Lecrenier <loic.lecrenier@me.com>
Co-authored-by: Tamo <tamo@meilisearch.com>
2023-01-11 18:15:32 +00:00
Tamo
6d658f4c52
fix a wrong error code + update some error messages
2023-01-11 19:14:11 +01:00
Tamo
bf573885ea
integrate the latest version of milli
2023-01-11 19:08:39 +01:00
Tamo
a68ac3a1dc
reformat the headers
2023-01-11 19:08:39 +01:00
Tamo
b252c87197
add tests on the sub settings routes
2023-01-11 19:08:39 +01:00
Loïc Lecrenier
b0b7ad7caf
Apply review suggestions
2023-01-11 19:08:39 +01:00
Loïc Lecrenier
c91ffec72e
Update Cargo.toml
2023-01-11 19:08:39 +01:00
Loïc Lecrenier
1fc11264e8
Refactor deserr integration
2023-01-11 19:08:39 +01:00
Loïc Lecrenier
2bc2e99ff3
Simplify declaration of the error codes
2023-01-11 19:08:39 +01:00
bors[bot]
808e184069
Merge #3324
...
3324: Add a test on the search route for each possible error codes r=irevoire a=irevoire
Co-authored-by: Tamo <tamo@meilisearch.com>
2023-01-11 16:08:19 +00:00
Tamo
7a30d98264
fix a flaky test
2023-01-11 14:54:29 +01:00
Tamo
d0a85057a3
fix the bad filter test
2023-01-11 11:37:12 +01:00
bors[bot]
b3574de809
Merge #3321
...
3321: Update the system http error code to return an internal server error r=irevoire a=irevoire
Fix parts of https://github.com/meilisearch/meilisearch/issues/3318
Co-authored-by: Tamo <tamo@meilisearch.com>
2023-01-11 10:27:13 +00:00
bors[bot]
59704c000c
Merge #3326
...
3326: Test error codes on settings r=irevoire a=irevoire
Co-authored-by: Tamo <tamo@meilisearch.com>
2023-01-11 10:07:52 +00:00
bors[bot]
b117c688f5
Merge #3328
...
3328: Replace published by released r=Kerollmops a=curquiza
Fix a bug introduced here: https://github.com/meilisearch/meilisearch/pull/3229
Regarding this line:
> * In multiple CIs: replace the `released` type by `published`, see [here](https://stackoverflow.com/questions/59319281/github-action-different-between-release-created-and-published ) why. Will not impact anything, but will prevent to fail our future automation
I made mistakes by replacing some un-relevant lines in the
- latest git workflow
- APT and brew workflow
-> the consequence was the workflow ran when releasing `rc0` but they shouldn't have. Luckily the check inside the workflow prevent any release.
<img width="1366" alt="Capture d’écran 2023-01-11 à 10 36 52" src="https://user-images.githubusercontent.com/20380692/211771382-d716ff16-0d53-41a9-90de-0d93e01e45fa.png ">
This fix is not mandatory thanks to the check inside the workflow, but I would rather roll back to avoid any issues when releasing the official v1 release.
Co-authored-by: curquiza <clementine@meilisearch.com>
2023-01-11 09:43:42 +00:00
curquiza
5ec85b7dfb
Replace published by released
2023-01-11 10:30:18 +01:00
bors[bot]
d80be0c28d
Merge #3322
...
3322: Update mini-dashboard to v0.2.5 r=curquiza a=mdubus
Co-authored-by: Morgane Dubus <30866152+mdubus@users.noreply.github.com>
2023-01-11 09:08:11 +00:00
Tamo
398c0c32cd
test all the error codes that can be throw in the settings
2023-01-10 18:19:27 +01:00
Tamo
d4157c0ce4
add a test on the search route for each possible error codes
...
snapshot the json directly instead of using the debug formatting
2023-01-10 17:59:24 +01:00
bors[bot]
98dffbf213
Merge #3317
...
3317: Remove the unused error codes r=irevoire a=irevoire
Remove some unused error code + fix the usage of the search+settings sort and filter error_code
Co-authored-by: Tamo <tamo@meilisearch.com>
2023-01-10 16:36:11 +00:00
Morgane Dubus
f63fee5e97
Update Cargo.toml
2023-01-10 15:11:25 +01:00
Tamo
f0d408c295
update the system http error code to return an internal server error
2023-01-10 14:33:46 +01:00
Tamo
d308684395
remove two ununsed error codes + fix the sort error_code
2023-01-10 11:32:11 +01:00
bors[bot]
e27bb8ab3e
Merge #3246
...
3246: Implement most of the error handling enhancement planned for v1.0 r=irevoire a=irevoire
Fix #3095 and #2325
Close https://github.com/meilisearch/meilisearch/pull/2540
Implements most of https://github.com/meilisearch/specifications/pull/212
## Generic error message we re-implements (in deserr):
- [x] Json
- [x] Incorrect value kind
- [x] Missing field
- [x] Unknown key
- [x] Unexpected
- [x] Reimplement the way we show the location
- [x] Query parameter
- [x] Incorrect value kind
- [x] Missing field
- [x] Unknown key
- [x] Unexpected
## Routes to implements:
- [x] Get search
- [x] Post search
- [x] Settings
- [x] Swap indexes
- [x] Task API
- [x] Documents ressource
Error codes to implements;
## Swap API
- [x] `duplicate_index_found` → `invalid_swap_duplicate_index_found`
## Search API
- [x] `invalid_search_q`
- [x] `invalid_search_offset`
- [x] `invalid_search_limit`
- [x] `invalid_search_page`
- [x] `invalid_search_hits_per_page`
- [x] `invalid_search_attributes_to_retrieve`
- [x] `invalid_search_attributes_to_crop`
- [x] `invalid_search_crop_length`
- [x] `invalid_search_attributes_to_highlight`
- [x] `invalid_search_show_matches_position`
- [x] `invalid_search_filter`
- [x] `invalid_search_sort`
- [x] `invalid_search_facets`
- [x] `invalid_search_highlight_pre_tag`
- [x] `invalid_search_highlight_post_tag`
- [x] `invalid_search_crop_marker`
- [x] `invalid_search_matching_strategy`
## Settings API
- [x] invalid_settings_displayed_attributes
- [x] invalid_settings_searchable_attributes
- [x] invalid_settings_filterable_attributes
- [x] invalid_settings_sortable_attributes
- [x] invalid_settings_ranking_rules
- [x] invalid_settings_stop_words
- [x] invalid_settings_synonyms
- [x] invalid_settings_distinct_attribute
- [x] Add invalid_settings_typo_tolerance
- [x] ~~invalid_settings_typo_tolerance_min_word_size_for_typos~~ (Merge in **invalid_settings_typo_tolerance**)
- [x] invalid_settings_faceting
- [x] invalid_settings_pagination
## Task API
- [x] invalid_task_date_filer → invalid_task_before_enqueued_at_filter (for all date filter) ?
## Document Resource
- [x] ~~`primary_key_inference_failed` → `index_primary_key_`~~ This doesn't exists anymore after `@dureuill` PR's on the primary key inference
------------------
# Changes
# `code` property
## Swap API
- [x] `invalid_swap_duplicate_index_found` ✅ [RENAME]
- [x] `invalid_swap_indexes` ✅ [NEW]
## Index API
### POST
- [x] `missing_index_uid` ✅ [NEW]
### POST/PATCH
- [x] `invalid_index_primary_key` ✅ [NEW]
### GET
- [x] `invalid_index_limit` ✅ [NEW]
- [x] `invalid_index_offset` ✅ [NEW]
## Documents API
### GET
- [x] `fields` parameter error `bad_request` → `invalid_document_fields` ✅ [NEW]
- [x] `limit` parameter error `bad_request` → `invalid_document_limit` ✅ [NEW]
- [x] `offset` parameter error `bad_request` → `invalid_document_offset` ✅ [NEW]
### POST/PUT
- [x] `?primaryKey` parameter error `bad_request` → `invalid_index_primary_key` ✅ [NEW]
## Keys API
### POST
- ~~`missing_parameter`~~
- [x] `missing_api_key_actions` ✅ [NEW]
- [x] `missing_api_key_indexes` ✅ [NEW]
- [x] `missing_api_key_expires_at` ✅ [NEW]
### GET
- [x] `limit` parameter `bad_request` → `invalid_api_key_limit` ✅ [NEW]
- [x] `offset` parameter `bad_request` → `invalid_api_key_offset` ✅ [NEW]
## Misc
- [x] ~~`invalid_geo_field`~~ → `invalid_document_geo_field` ✅ [RENAME]
# `type` property
## `system` ✅ [NEW]
- [x] `no_space_left_on_device` error code
- [x] `io_error` error code (**does not exist in the current spec, need a catch-up**)
- [x] `too_many_open_files` error code (**does not exist in the current spec, need a catch-up**)
Co-authored-by: Tamo <tamo@meilisearch.com>
Co-authored-by: Loïc Lecrenier <loic.lecrenier@me.com>
2023-01-09 16:25:48 +00:00
Tamo
ff843881c5
remove the documentation of the query parameter extractor module
2023-01-09 15:14:48 +01:00
Loïc Lecrenier
ae08fba76e
Remove forgotten comment
2023-01-09 13:45:03 +01:00
Loïc Lecrenier
af6d4b3031
Remove unused deserr extractor
2023-01-09 13:43:16 +01:00
Tamo
b03ee54fe0
makes clippy turbo-happy
2023-01-09 13:04:31 +01:00
Tamo
d17efb9ed6
use the published version of deserr
2023-01-09 12:51:10 +01:00
Loïc Lecrenier
9ab791bedc
Update error codes on the api key routes
2023-01-09 12:30:25 +01:00
Loïc Lecrenier
96105a5e8d
Update error codes on the documents/ routes
2023-01-09 12:30:25 +01:00