Skip to content

added api keys and authentication via http head#9

Open
zfoster wants to merge 1 commit intosouthpolesteve:masterfrom
zfoster:api_keys
Open

added api keys and authentication via http head#9
zfoster wants to merge 1 commit intosouthpolesteve:masterfrom
zfoster:api_keys

Conversation

@zfoster
Copy link

@zfoster zfoster commented Apr 21, 2013

is this the kind of thing you're looking for?

@southpolesteve
Copy link
Owner

Ah not really. A few things.

  1. We're gonna have to write some test :) Testing for murfie not so important, but testing for libs like this is huge. People will use it in all kinds of ways and we need to know it will do what we expect it too.
  2. Check out how active admin does this: http://activeadmin.info/docs/1-general-configuration.html#authentication

Basically they just leave it up to the user to define their auth method. I was thinking something similar here

@zfoster
Copy link
Author

zfoster commented Apr 23, 2013

I figured it wasn't ideal haha. It was good to get that all written down
though to learn how to do it that way. I'll check out the active admin one
and send another PR for review. I don't really understand what's going on
with travis so I might wait to do some testing with you

On Mon, Apr 22, 2013 at 9:23 PM, Steve Faulkner notifications@github.comwrote:

Ah not really. A few things.

We're gonna have to write some test :) Testing for murfie not so
important, but testing for libs like this is huge. People will use it in
all kinds of ways and we need to know it will do what we expect it too.
2.

Check out how active admin does this:
http://activeadmin.info/docs/1-general-configuration.html#authentication

Basically they just leave it up to the user to define their auth method. I
was thinking something similar here


Reply to this email directly or view it on GitHubhttps://github.com//pull/9#issuecomment-16835632
.

Zachary Foster
zach@murfie.com
1 S Pinckney St Ste 818
Madison WI 53703
World's largest new & used CD store... with a twist.
Still got CDs? Send them to Murfie https://www.murfie.com/about_kits,
free!- murfie.com | twitter http://twitter.com/murfiemusic |
facebookhttp://facebook.com/murfiemusic

@zfoster
Copy link
Author

zfoster commented Jun 18, 2013

can we go through what coding this up would be like? It's still slightly nebulous, I'm not sure I can navigate it.

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.

2 participants