-
Bug
-
Resolution: Fixed
-
Neutral
-
2.2
-
None
-
None
-
-
Empty show more show less
-
DevX 35
-
2
-
Yes
If the lang param is invalid, I18nContainerRequestFilter throws BadRequestException, however, this exception is swallowed by PreMatchContainerRequestContext. The class expects the exception should be a child of IOException.
public class I18nContainerRequestFilter implements ContainerRequestFilter { private Locale determineLocaleFromQueryParameter(MultivaluedMap<String, String> queryParameters) { Â Â ... Â Â if (!isValidLanguageTag(localeString)) { Â Â Â Â throw new BadRequestException(String.format("Language parameter: %s is not allowed", localeString)); Â Â } Â Â ... } }
And
class PreMatchContainerRequestContext ...{ public synchronized BuiltResponse filter() { Â Â RESTEasyTracingLogger tracingLogger = RESTEasyTracingLogger.getInstance(this.httpRequest); Â Â long totalTimestamp = tracingLogger.timestamp("REQUEST_FILTER_SUMMARY");Â Â while(this.requestFilters != null && this.currentFilter < this.requestFilters.length) { Â Â Â Â ContainerRequestFilter filter = this.requestFilters[this.currentFilter++];Â Â Â Â try { Â Â Â Â Â Â this.suspended = false; Â Â Â Â Â Â this.response = null; Â Â Â Â Â Â this.throwable = null; Â Â Â Â Â Â this.inFilter = true; Â Â Â Â Â Â long timestamp = tracingLogger.timestamp("REQUEST_FILTER"); Â Â Â Â Â Â filter.filter(this); Â Â Â Â Â Â tracingLogger.logDuration("REQUEST_FILTER", timestamp, new Object[]{filter}); Â Â Â Â } catch (IOException var10) { Â Â Â Â Â Â throw new ApplicationException(var10); Â Â Â Â } finally { Â Â Â Â Â Â this.inFilter = false; Â Â Â Â } ... }
Acceptance criteria
- is related to
-
PAGES-588 Language tag in spa renderer misses the country in the url
- Closed
1.
|
Implementation | Completed | Marek Strucka | |
2.
|
Review | Completed | Dai Ha | |
3.
|
PiQA | Completed | Dai Ha | |
4.
|
QA | Completed | Milan Divilek |