http URL validation is too restrictive about abrader-gms HOT 11 CLOSED

example error for people who search

Error: Failed to apply catalog: Parameter webhook_url failed on Git_webhook[puppet_webhook]: Git webhook URL must be fully qualified, not 'https://puppet:[email protected]:8089/payload' at /etc/puppetlabs/puppet/environments/production/site/profile/manifests/master.pp:12
Wrapped exception:
Git webhook URL must be fully qualified, not 'https://puppet:[email protected]:8089/payload'

from abrader-gms.

Weakened the validation per your suggest, as seen in this commit: 61def08

from abrader-gms.

Or...rather this commit: 9ae51c1

from abrader-gms.

yeah I found it ;)

from abrader-gms.

I am still getting error:

Failed to apply catalog: Parameter webhook_url failed on Git_webhook[web_post_receive_webhook]: Git webhook URL must be fully qualified, not 'puppet:[email protected]:8088/payload' at /etc/puppetlabs/puppet/environments/production/modules/profiles/manifests/r10k.pp:113 Wrapped exception: Git webhook URL must be fully qualified, not 'puppet:[email protected]:8088/payload'

• zack/r10k, 2.6.5
• abrader/gms, 0.0.6

r10k code

 git_webhook { 'web_post_receive_webhook' :
    ensure             => present,
    webhook_url        => 'https://puppet:[email protected]:8088/payload',
    token              =>  hiera('gitlab_api_token'),
    project_name       => 'puppet/control',
    server_url         => 'https://fqdn.gitlab.com',
    disable_ssl_verify => true,
    provider           => 'gitlab',
}

Shouldn't matter but, I'm using gitlab.

from abrader-gms.

My apologies for the delay. Confirmed the issue. Thanks for the submission. I will be getting to the refactoring of the regex shortly.

from abrader-gms.

@oznah Please try the validation from this release: https://github.com/abrader/abrader-gms/blob/master/lib/puppet/type/git_webhook.rb#L36

Worked for me, would like your input.

from abrader-gms.

I will give it a try tomorrow.

On Thu, Apr 16, 2015, 3:57 PM Andrew Brader [email protected]
wrote:

@oznah https://github.com/oznah Please try the validation from this
release:
https://github.com/abrader/abrader-gms/blob/master/lib/puppet/type/git_webhook.rb#L36

Worked for me, would like your input.

—
Reply to this email directly or view it on GitHub
#5 (comment).

from abrader-gms.

Confirmed with other interested parties this is now working for them as well. Committed changes to a new forge release v.0.0.8: https://forgeapi.puppetlabs.com/v3/files/abrader-gms-0.0.8.tar.gz

from abrader-gms.

Sorry for the delay. I confirmed that this is working now. No more error.

from abrader-gms.

No worries. Thanks for checking.

from abrader-gms.