JessYanCoding / JessYanCoding/ProgressManager

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

Aperta
#67 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Java
Stelle
2.7k
Fork
289
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.