Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

Add method for validating ReportingContext #2

Closed
glennblock opened this issue Dec 1, 2014 · 2 comments
Closed

Add method for validating ReportingContext #2

glennblock opened this issue Dec 1, 2014 · 2 comments

Comments

@glennblock
Copy link

The o365 Modular Input for Splunk invokes O365 using this library. It requires a user to provide credentials as well as a report name. Validation doesn't appear to happen until you attempt to Stream. It would be nice to have a simple validation method that can verify whether or not the credentials / report is valid without having to stream.

Perhaps there is a way to do this and it is just not obvious (to me).

@glennblock glennblock changed the title Add method for validating RepotingContext Add method for validating ReportingContext Dec 1, 2014
@julienjtpierre
Copy link
Member

1223e17
Added ValidateAccessToReport method in the ReportingStream class.
The reason why I added the method in the ReportingStream and not in the ReportingContext is because I want to validate the user credentials + report access, not only user credentials.
We could potentially another validate method in the ReportingContext later

@glennblock
Copy link
Author

Awesome.
On Tue, Dec 2, 2014 at 8:28 PM Julien Pierre notifications@github.com
wrote:

1223e17
1223e17
Added ValidateAccessToReport method in the ReportingStream class.
The reason why I added the method in the ReportingStream and not in the
ReportingContext is because I want to validate the user credentials +
report access, not only user credentials.
We could potentially another validate method in the ReportingContext later


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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants