Skip to content
This repository has been archived by the owner on Aug 14, 2018. It is now read-only.

Added metadata preset for Sign4 #44

Closed
wants to merge 1 commit into from
Closed

Added metadata preset for Sign4 #44

wants to merge 1 commit into from

Conversation

dcylabs
Copy link

@dcylabs dcylabs commented Jun 28, 2017

No description provided.

@mdwhatcott
Copy link
Contributor

@dcylabs - Can you tell us more about what you're trying to accomplish?

@dcylabs
Copy link
Author

dcylabs commented Jun 29, 2017

@mdwhatcott - I try using Sign4 behind CloudFlare, where the guess logic of service and region can't work. With this PR you can hard-pass a preset of metadata that could not be guessed

Service string
}

func (m *Metadata) SetAlgorithm(value string){
Copy link
Contributor

Choose a reason for hiding this comment

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

I see that these setter functions prevent any original value from being overwritten--what's the need for that constraint?

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm wondering why we need setter functions at...

@mdwhatcott
Copy link
Contributor

Due to #49 I am closing all pull requests.

@mdwhatcott mdwhatcott closed this Aug 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants