Skip to content
This repository has been archived by the owner on Jun 4, 2021. It is now read-only.

Make githubsource's SecretToken optional #297

Closed
duglin opened this issue Mar 27, 2019 · 6 comments
Closed

Make githubsource's SecretToken optional #297

duglin opened this issue Mar 27, 2019 · 6 comments

Comments

@duglin
Copy link

duglin commented Mar 27, 2019

Right now SecretToken appears to be required:
https://github.com/knative/eventing-sources/blob/master/pkg/apis/sources/v1alpha1/githubsource_types.go#L66

I think it would be nice if it were made optional, and if not provided then the system just created something to use (like a UUID). This value has no meaning outside of this subscription so I don't need the requirement to force the user to specify it. However, if someone does want to do so they can.

@n3wscott
Copy link
Contributor

gcppubsubsource has been removed from this repo. Please see http://github.com/google/knative-gcp

PullSubscription is the replacement.

/close

@knative-prow-robot
Copy link
Contributor

@n3wscott: Closing this issue.

In response to this:

gcppubsubsource has been removed from this repo. Please see http://github.com/google/knative-gcp

PullSubscription is the replacement.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@duglin
Copy link
Author

duglin commented Apr 3, 2020

@n3wscott I think you closed this one by mistake - this isn't about gcp

@github-actions
Copy link

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.

@github-actions github-actions bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 11, 2020
@duglin
Copy link
Author

duglin commented Oct 11, 2020

/remove-lifecycle stale

@knative-prow-robot knative-prow-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 11, 2020
@lionelvillard
Copy link
Member

move to knative-sandbox/eventing-github: knative-extensions/eventing-github#36

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

No branches or pull requests

4 participants