Skip to content

Commit cc091c4

Browse files
Fixed parsing of the connect string when using the Azure Config Store
Provider.
1 parent 03182f3 commit cc091c4

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
@@ -36,6 +36,9 @@ Common Changes
3636
(`issue 493 <https://github.com/oracle/python-oracledb/issues/493>`__).
3737
#) Miscellaneous grammar and spelling fixes by John Bampton
3838
(`PR 479 <https://github.com/oracle/python-oracledb/pull/479>`__).
39+
#) Fixed parsing of the connection string in the
40+
:ref:`Azure App Centralized Configuration Provider
41+
<azureappstorageprovider>`.
3942
#) Fixed memory corruption in DataFrame queries
4043
(`issue 489 <https://github.com/oracle/python-oracledb/issues/489>`__).
4144
#) 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)