Skip to content

Conversation

@MarkGibbons
Copy link
Member

No description provided.

markgibbons added 5 commits May 28, 2020 12:25
Add a slash to the end of the  base url
The option default is false. We will fix the base url by default
Handle chef server URL specification without a trailing /
Calculate the URL to use for global endpoint requests
Add an option to the configuration to use the URL as specified rather
than adding forward slashes and computing the global base.

This pull request completes mile stone 1.0.  All of the chef server
endpoints and methods for chef server 11 are defined.  Most have
been updated for chef server 12.
@MarkGibbons
Copy link
Member Author

This is a breaking change. By default the chef server URL will have a / slash added if none is present. If an organization is specified in the URL the global endpoint will be be computed for the functions that need to address the base chef server URL.

@MarkGibbons
Copy link
Member Author

MarkGibbons commented Jun 2, 2020

@mudash @vsingh-msys If you get a moment. Does this one make sense to you?

@kvivek1115
Copy link
Contributor

kvivek1115 commented Jun 3, 2020

@MarkGibbons Looks good to me.

The only concern about additional param UseOrg & UseGlobal are passed to decide about URL construct.

If we can construct based on
a) URL is relative if passed without forwarding slash "clients", "cookbooks" etc.

b) URL is absolute if passed with forwarding slash "/status", "/association_requests" etc.

that would be helpful also global URL looks much similar to chef_server_root of chef infra config.rb option.

chef_server_root 'https://localhost'
chef_server_url  'https://localhost/organizations/4thcoffee'

@MarkGibbons
Copy link
Member Author

@vsingh-msys thanks for the comments. I'll take a look at using the absolute path name for the global requests. I added the UseGlobal and UseOrg to the parameter list because specifying the value explicitly seemed to make it clear which level the call was functioning on.

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.

3 participants