JessYanCoding / JessYanCoding/ProgressManager

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

Offen
#67 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Java
Sterne
2.7k
Forks
289
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

### 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.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.