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:<access_token>@ 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 <#21 (comment)>, 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.