github urls with access token are not supported

[godfryd]

Such urls has this form: https://\<access-token>@github.com/.... For example: https://fiuahjrehfakhdhauwe@github.com/nephila/giturlparse

[yakky]

@godfryd thanks for reporting I need to go back to work on #19 where we should be able to easily add this

[protoroto]

@godfryd Could you try on latest version? I'm able to parse that url correctly now

[godfryd]

Hello,

I tried giturparse 0.11.1 on the URL: https://fiuahjrehfakhdhauwe@github.com/nephila/giturlparse

I run:

p=giturlparse.parse('https://fiuahjrehfakhdhauwe@github.com/nephila/giturlparse')
print(p.data)

Parsed data does not seem well:

{'port': '',
 'path_raw': '',
 'groups_path': '',
 'owner': 'nephila',
 '_user': 'git',
 'protocols': ['https'],
 'protocol': 'https',
 'domain': 'fiuahjrehfakhdhauwe@github.com',
 'pathname': '/nephila/giturlparse',
 'repo': 'giturlparse',
 'path': '',
 'branch': '',
 'url': 'https://fiuahjrehfakhdhauwe@github.com/nephila/giturlparse',
 'platform': 'gitlab'}

I would expect that platform is github not gitlab, domain seems to be incorrect, access token could be stored in user field.

[protoroto]

@godfryd You're right: I guess access tokens are not supported yet. I'll look into it

[protoroto]

@godfryd I'm digging into this: it looks like urls with access token like the one you mentioned https://<access_token>@github.com/nephila/giturlparse are not supported by github/gitlab . I'm seeing that urls like https://oauth2:<access_token>@github.com/nephila/giturlparse are instead valid and supported (both for gitlab and github), so I'll work on this feature.

[godfryd]

ok, thanks :)

Michał Nowikowski

On Thu, Aug 31, 2023 at 11:48 AM Leonardo Cavallucci < @.***> wrote:

@godfryd https://github.com/godfryd I'm digging into this: it looks like urls with access token like the one you mentioned https:// @.***/nephila/giturlparse are not supported by github/gitlab . I'm seeing that urls like https://oauth2:@ github.com/nephila/giturlparse are instead valid and supported (both for gitlab and github), so I'll work on this feature.

— Reply to this email directly, view it on GitHub https://github.com/nephila/giturlparse/issues/21#issuecomment-1700719958, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABLDNZIJ2MO3JD7YI2YCMTXYBMXTANCNFSM4KAIPR7Q . You are receiving this because you were mentioned.Message ID: @.***>

[protoroto]

@godfryd Hey! We've just release 0.12.0 that should support those urls.