Skip to content

Commit 1337408

Browse files
Fixed parsing of the connect string when using the Azure Config Store
Provider.
1 parent 6466c42 commit 1337408

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

doc/src/release_notes.rst

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,9 @@ Common Changes
3939
(`issue 493 <https://github.com/oracle/python-oracledb/issues/493>`__).
4040
#) Miscellaneous grammar and spelling fixes by John Bampton
4141
(`PR 479 <https://github.com/oracle/python-oracledb/pull/479>`__).
42+
#) Fixed parsing of the connection string in the
43+
:ref:`Azure App Centralized Configuration Provider
44+
<azureappstorageprovider>`.
4245
#) Fixed memory corruption in DataFrame queries
4346
(`issue 489 <https://github.com/oracle/python-oracledb/issues/489>`__).
4447
#) Added support for converting an OracleDataFrame object to a foreign data

src/oracledb/plugins/azure_config_provider.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -188,9 +188,10 @@ def _parse_parameters(protocol_arg: str) -> dict:
188188
parameters = {
189189
key.lower(): value[0] for key, value in parsed_values.items()
190190
}
191-
parameters["appconfigname"] = (
192-
protocol_arg[:pos].rstrip("/").rstrip(".azconfig.io") + ".azconfig.io"
193-
)
191+
config_name = protocol_arg[:pos].rstrip("/")
192+
if not config_name.endswith(".azconfig.io"):
193+
config_name += ".azconfig.io"
194+
parameters["appconfigname"] = config_name
194195
return parameters
195196

196197

0 commit comments

Comments
 (0)