Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CredentialStore methods should declare throws IOException #40

Closed
wonderfly opened this issue Jan 9, 2015 · 2 comments
Closed

CredentialStore methods should declare throws IOException #40

wonderfly opened this issue Jan 9, 2015 · 2 comments
Assignees
Labels
priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@wonderfly
Copy link
Contributor

From yan...@google.com on May 30, 2012 09:23:01

External references, such as a standards document, or specification? http://javadoc.google-oauth-java-client.googlecode.com/hg/1.9.0-beta/com/google/api/client/auth/oauth2/CredentialStore.html Java environments (e.g. Java 6, Android 2.3, App Engine, or All)? All Please describe the feature requested. Unfortunately, it's going to be hard to implement FileCredentialStore because each of the methods require I/O access. We should really declare throws IOException on this for general use. That also means updating all of the callers.

Original issue: http://code.google.com/p/google-oauth-java-client/issues/detail?id=40

@wonderfly wonderfly added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. imported priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Jan 9, 2015
@wonderfly wonderfly self-assigned this Jan 9, 2015
@wonderfly
Copy link
Contributor Author

From rmis...@google.com on May 30, 2012 13:20:11

http://codereview.appspot.com/6244065/

Status: Started

@wonderfly
Copy link
Contributor Author

From rmis...@google.com on May 31, 2012 04:32:36

Status: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Development

No branches or pull requests

1 participant