[BUG] RemoteConfig unit tests fail in non-English (e.g., Korean) Locales due to Date Parsing
Maintainer antworten meist innerhalb von 4 Tagen
Bewertung
Dieses Issue wurde noch nicht bewertet.
Beschreibung
[REQUIRED] Step 2: Describe your environment
- Operating System version: macOS (Apple Silicon, ko_KR locale)
- Firebase SDK version: 9.7.0 (Current development branch)
- Library version: N/A
- Firebase Product: Remote Config
[REQUIRED] Step 3: Describe the problem
Steps to reproduce:
The unit tests for Remote Config fail when the system's default locale is set to a non-English language (e.g., Korean). This is due to RemoteConfigUtil using SimpleDateFormat without an explicit Locale.ENGLISH, causing parsing errors or string comparison mismatches against English date formats.
Error Output:
org.junit.ComparisonFailure: expected:<...NODE","updateTime":"[Tue, 08 Dec] 2020 15:49:51 UTC",...>
but was:<...NODE","updateTime":"[화, 08 12월] 2020 15:49:51 UTC",...>
Caused by: java.text.ParseException: Unparseable date: "Tue, 08 Dec 2020 15:49:51 UTC"
at java.base/java.text.DateFormat.parse(DateFormat.java:427)
at com.google.firebase.remoteconfig.RemoteConfigUtil.convertFromUtcDateFormat(RemoteConfigUtil.java:91)
Relevant Code:
The issue occurs in RemoteConfigUtil.java. There are 4 methods using SimpleDateFormat without an explicit Locale. To ensure consistent behavior across different system locales, all instances of SimpleDateFormat should be instantiated with Locale.ENGLISH.
Affected Methods:
convertToUtcZuluFormat(long millis)convertToUtcDateFormat(long millis)convertFromUtcZuluFormat(String dateString)convertFromUtcDateFormat(String dateString)
Current implementation:
SimpleDateFormat dateFormat = new SimpleDateFormat(PATTERN);
Recommended fix:
SimpleDateFormat dateFormat = new SimpleDateFormat(PATTERN, Locale.ENGLISH);
I've already identified the fix and verified it locally by setting the locale to Korean. I'd like to submit a Pull Request to resolve this issue.
- Vorherrschende Sprache
- Java
- Sterne
- 620
- Forks
- 306
- Ø Merge
- 4 T. 7 Std.
- Gemergte PRs (30 T.)
- 13
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus firebase/firebase-admin-java
-
google-http-client-jackson2 cannot be excluded even when a JsonFactory is configured via FirebaseOptionsEvtl. vergeben @lahirumaramba hat das vor 6 Tagen übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
firebase/firebase-admin-java#1231 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Auth list tenants is not pageableEvtl. vergeben @siddharthmohan619-eng hat das vor 209 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
firebase/firebase-admin-java#1192 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 4 Tagen
-
needs-triage type: feature request
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
firebase/firebase-admin-java#1149 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 4 Tagen
-
needs-triage type: feature request
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 45/100
firebase/firebase-admin-java#1142 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Opentelemetry-context conflict, please bump google-api-client dependencyEvtl. wieder frei @lahirumaramba hat das vor 335 Tagen übernommen, und es ist kein Pull Request offen. Offenapi: core
firebase/firebase-admin-java#1138 · 8 Kommentare · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 4 Tagen
Alle Issues in firebase/firebase-admin-java
Ähnliche Issues
-
CalendarEventAttendance/get returns eventAttendanceStatus while the doc says attendanceStatusEvtl. vergeben @chibenwa hat das heute übernommen. Offenbug claude
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
linagora/tmail-backend#2697 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
apache/skywalking#14120 ·
Maintainer antworten meist innerhalb von 1 Tag
-
[BUG] Case-insensitive search suggestions miss items when the JVM default locale is TurkishEvtl. vergeben @thswlsqls hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
[Feature] 关于启动游戏进度条显示的优化Offenenhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
HMCL-dev/HMCL#6943 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
NameAllocator generates colliding identifiers with ignorable charactersEvtl. vergeben @PHJ2000 hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
Maintainer antworten meist innerhalb von 1 Tag