diff options
| author | Bryan Newbold <bnewbold@robocracy.org> | 2019-04-18 18:01:33 -0700 |
|---|---|---|
| committer | Bryan Newbold <bnewbold@robocracy.org> | 2019-04-18 18:01:33 -0700 |
| commit | 71ffc180036088d310714ca3d960b6378290a809 (patch) | |
| tree | 3fef3cabb3e208575cc0e27e12920ea8d8f1fa89 | |
| parent | 11dfac5f8f9ced9b56cf277d0e3adeccc572b251 (diff) | |
| download | fatcat-71ffc180036088d310714ca3d960b6378290a809.tar.gz fatcat-71ffc180036088d310714ca3d960b6378290a809.zip | |
rust: API lower-cases DOI lookups
| -rw-r--r-- | rust/src/endpoint_handlers.rs | 6 | ||||
| -rw-r--r-- | rust/tests/test_api_server_http.rs | 11 |
2 files changed, 15 insertions, 2 deletions
diff --git a/rust/src/endpoint_handlers.rs b/rust/src/endpoint_handlers.rs index d9bd3403..f43b3559 100644 --- a/rust/src/endpoint_handlers.rs +++ b/rust/src/endpoint_handlers.rs @@ -277,10 +277,12 @@ impl Server { jstor_id, ) { (Some(doi), None, None, None, None, None, None, None) => { - check_doi(doi)?; + // DOIs always stored lower-case; lookups are case-insensitive + let doi = doi.to_lowercase(); + check_doi(&doi)?; release_ident::table .inner_join(release_rev::table) - .filter(release_rev::doi.eq(doi)) + .filter(release_rev::doi.eq(&doi)) .filter(release_ident::is_live.eq(true)) .filter(release_ident::redirect_id.is_null()) .first(conn)? diff --git a/rust/tests/test_api_server_http.rs b/rust/tests/test_api_server_http.rs index 66f36a14..c891ed2e 100644 --- a/rust/tests/test_api_server_http.rs +++ b/rust/tests/test_api_server_http.rs @@ -362,6 +362,17 @@ fn test_lookups() { Some("bigger example"), ); + // not lower-case + helpers::check_http_response( + request::get( + "http://localhost:9411/v0/release/lookup?doi=10.123/ABC", + headers.clone(), + &router, + ), + status::Ok, + Some("bigger example"), + ); + // URL encoded helpers::check_http_response( request::get( |