JessYanCoding / JessYanCoding/ProgressManager

Progress Listener can't listen to url with "/" query

オープン
#67 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Java
スター
2.7k
フォーク
289
PR マージ指標
30日以内にマージされた PR はありません

説明

### Problem

Progress Listener cannot listen to request with query that contains "/", because mResponseListeners Hashmap failed to detect item, when requested "/" change into "%2F", so containsKey(key) will return false

ex.

```kotlin
https://localhost/v3.1?type_file=pdf&start_date=2022/01/07&end_date=2022/02/06
```

the start_date and end_date will cause Problem.

#### Already Done

I've already tried to replace "/" before request with "%2F" but it reversed now in the HashMap key it has "/" but the checked key contains "%2F", so it still return false.

#### Potential Problem

Please check on ProgressManager.java on line 259

#### Potential Fix

Implement more complex search algorithm for checking key, or custom check on HashMap key.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。