Skip to content
This repository was archived by the owner on Mar 25, 2024. It is now read-only.

When setting up dogapi you need to give the application key also#60

Merged
clofresh merged 1 commit into
DataDog:masterfrom
zupo:patch-1
May 7, 2014
Merged

When setting up dogapi you need to give the application key also#60
clofresh merged 1 commit into
DataDog:masterfrom
zupo:patch-1

Conversation

@zupo

@zupo zupo commented Aug 22, 2013

Copy link
Copy Markdown
Contributor

No description provided.

@miketheman

Copy link
Copy Markdown
Contributor

Hi @zupo !
From our docs:

Requests that read data require full access and also require an application key.

Since the typical use case is to submit metrics and events, could you explain a bit on the need for the read aspect form fabric?

@zupo

zupo commented Aug 22, 2013

Copy link
Copy Markdown
Contributor Author

Not sure I understand your question?

@miketheman

Copy link
Copy Markdown
Contributor

Sorry - I was asking what use case have you seen where the application key is required?

@clofresh

clofresh commented May 5, 2014

Copy link
Copy Markdown
Contributor

For the fabric integration itself, you don't need to set the application key to be able to submit data, but if you want to later use the http api connection for other things that do need the application key, it would be nice to have it set.

@zupo For backwards compatibility, and since the application key is optional for the main fabric use case, could you make that application_key default to None?

@zupo

zupo commented May 5, 2014

Copy link
Copy Markdown
Contributor Author

Sure.

clofresh added a commit that referenced this pull request May 7, 2014
When setting up dogapi you need to give the application key also
@clofresh
clofresh merged commit 48e8a06 into DataDog:master May 7, 2014
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.

3 participants