Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add-instace-support #157

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add-instace-support #157

wants to merge 5 commits into from

Conversation

kouylekov-usit
Copy link
Contributor

@kouylekov-usit kouylekov-usit commented Oct 24, 2024

This commit add an instance support for TACL. You can user instances
with api_key for a client that is allowed instances authentication

example:

tacl p11 --api-key <api_key> --link-id <> --upload myfile

This pull request resolves #104

This commit add an instance support for TACL. You can user instances
with api_key for a client that is allowed instances authentication

example:

tacl p11 --api-key <api_key> --link-id <> --upload myfile
Copy link

Triggered from #157 by @​kouylekov-usit.

Checking if we can fast forward master (d306615) to add-instace-support (0731844).

Target branch (master):

commit d306615f6d8c8b8f280d0f8308bfff6019d8952b (HEAD -> master, origin/master, origin/HEAD)
Author: Eirik Haatveit <[email protected]>
Date:   Tue Sep 17 15:25:18 2024 +0200

    Revert "ci(docs): add automatic generation of CHANGELOG"
    
    This reverts commit 32e2558c3ef77ab8da335b42134533697d5c3938.
    
    As the commits generated by CI fail branch protection rules, this will
    have to wait until we figure out a good pattern for allowing automated
    PR generation and merging of changelog updates.

Pull request (add-instace-support):

commit 07318449910be4dae7c22902694c348f8835f930 (pull_request/add-instace-support)
Author: Milen Kouylekov <[email protected]>
Date:   Thu Oct 24 16:39:24 2024 +0200

    feat: Add instance support to tacl
    
    This commit add an instance support for TACL. You can user instances
    with api_key for a client that is allowed instances authentication
    
    example:
    
    tacl p11 --api-key <api_key> --link-id <> --upload myfile

It is possible to fast forward master (d306615) to add-instace-support (0731844). If you have write access to the target repository, you can add a comment with /fast-forward to fast forward master to add-instace-support.

Copy link

Triggered from #157 by @​kouylekov-usit.

Checking if we can fast forward master (d306615) to add-instace-support (fb2fe5c).

Target branch (master):

commit d306615f6d8c8b8f280d0f8308bfff6019d8952b (HEAD -> master, origin/master, origin/HEAD)
Author: Eirik Haatveit <[email protected]>
Date:   Tue Sep 17 15:25:18 2024 +0200

    Revert "ci(docs): add automatic generation of CHANGELOG"
    
    This reverts commit 32e2558c3ef77ab8da335b42134533697d5c3938.
    
    As the commits generated by CI fail branch protection rules, this will
    have to wait until we figure out a good pattern for allowing automated
    PR generation and merging of changelog updates.

Pull request (add-instace-support):

commit fb2fe5cd13727fc9c81edf0b73dd4f5221a843f8 (pull_request/add-instace-support)
Author: Milen Kouylekov <[email protected]>
Date:   Fri Oct 25 15:38:51 2024 +0200

    Add support for remote-path in uploads and downloads

It is possible to fast forward master (d306615) to add-instace-support (fb2fe5c). If you have write access to the target repository, you can add a comment with /fast-forward to fast forward master to add-instace-support.

if match:=re.compile(patten).match(link_id):
link_id = uuid.UUID(match.group("link_id"))
if match.group("instance_type") == "c" and not secret_challenge:
click.echo("instance authentication requires a secret challenge")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could consider interactively asking for the secret here

@click.option(
'--remote-path',
required=False,
help='Specify a path on the remote server to upload'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't it for both upload and download?

'--secret-challenge',
required=False,
default=None,
help='Pass a secret challenge for instance authentication'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should insist on passing it via a file here, to avoid it being in the PID list

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support using import links
2 participants