keboola / tde-exporter

export storage api tables to TDE
MIT License
2 stars 0 forks source link

fix connection url composition #38

Closed kacurez closed 1 year ago

kacurez commented 1 year ago

debug https://keboolaglobal.slack.com/archives/C0553EM2RGX/p1683646594611759 Na zendesku som narazil na to ze zakaznik ktory pusta TDE exporter na starej fronte, mu potom spadne upload do Tableau servru ale uz hned na prvom api call na tokens verify.

{"error":"Argument \\"id\\" is expected to be type \\"int\\", value \\"verify\\" given.","exceptionId":"kbc-us-east-1-connection-87e84e9a02e9c74967135270c1deecf2"}'

To bude zrejme nedavnou zmenou na connection kde uz nepodporuju double slash v ceste, "[GET] [//v2/storage/tokens/verify]", Problem je zrejme v tom ze v connectionIndexUrl = os.environ['KBC_URL'] + '/v2/storage' KBC_URL obsahuje slash na konci. Nepodarilo sa mi to overit (pustil som kontajner len na novej fronte a tam to nespadlo) ale pre istotu skusim takyto naslepo fix ktory by mal tento pripad poriesit.

kacurez commented 1 year ago

aha tak spadlo to na testoch,

    from urllib.parse import urljoin
E   ImportError: No module named parse

(to riesenie mam od ChatGPT) :)

kacurez commented 1 year ago

uz som to fixol, lokalne to skladanie url funguje tak ako predpokladam

image