Skip to content

Commit 1d20b2a

Browse files
committed
fix: resolve double URL-encoding and add singular NIFTY FIN SERVICE mapping alias
1 parent 1bb7692 commit 1d20b2a

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

am-market-data/market-data-provider/src/main/java/com/am/marketdata/provider/upstox/UpstoxIndexIdentifier.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@ public void initialize() {
5858
// Add custom aliases for UI symbols that differ from Upstox instrument keys
5959
indexKeyMap.put("NIFTY PHARMACEUTICALS", "NSE_INDEX|Nifty Pharma");
6060
indexKeyMap.put("NIFTY FIN SERVICES", "NSE_INDEX|Nifty Fin Service");
61+
indexKeyMap.put("NIFTY FIN SERVICE", "NSE_INDEX|Nifty Fin Service");
6162
indexKeyMap.put("NIFTY CONSR DURABLE", "NSE_INDEX|NIFTY CONSR DURBL");
6263

6364
log.info("Loaded {} Upstox Index Identifiers", indexKeyMap.size());

am-market-data/market-data-provider/src/main/java/com/am/marketdata/provider/upstox/UpstoxSdkService.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -344,7 +344,9 @@ private com.am.marketdata.provider.upstox.model.HistoricalDataResponse fetchHist
344344
String normalizedKey, String unit, Integer interval, String toDate, String fromDate, boolean isQueryingToday) {
345345
try {
346346
String token = getDynamicAccessToken();
347-
String encodedKey = java.net.URLEncoder.encode(normalizedKey, java.nio.charset.StandardCharsets.UTF_8.toString())
347+
// Safeguard against double-encoding (decodes the key to plain text first, then encodes it cleanly exactly once)
348+
String decodedKey = java.net.URLDecoder.decode(normalizedKey, java.nio.charset.StandardCharsets.UTF_8.toString());
349+
String encodedKey = java.net.URLEncoder.encode(decodedKey, java.nio.charset.StandardCharsets.UTF_8.toString())
348350
.replace("+", "%20");
349351

350352
String urlStr;

0 commit comments

Comments
 (0)