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

Unable to call LoggingApi.entries.tail() #407

Open
schweitzer-churned opened this issue May 19, 2022 · 11 comments
Open

Unable to call LoggingApi.entries.tail() #407

schweitzer-churned opened this issue May 19, 2022 · 11 comments

Comments

@schweitzer-churned
Copy link

schweitzer-churned commented May 19, 2022

When I try to use the tail function provided in the Logging API, I am getting the following error

[{ "error": { "code": 400, "message": "Invalid value (Object), ", "status": "INVALID_ARGUMENT", "details": [ { "@type": "type.googleapis.com/google.rpc.BadRequest", "fieldViolations": [ { "description": "Invalid value (Object), " } ] } ] } } ]

I stumbled across this issue and I am assuming this is a similar bug. Can you confirm? And are there plans to use the gRPC versions to enable streaming into flutter?

@kevmoo
Copy link
Collaborator

kevmoo commented May 19, 2022

Exactly. We cannot do streaming APIs

@postmeridiem
Copy link

Thanks for confirming! I guess this function should also be removed from discovery at some point.

@schweitzer-churned
Copy link
Author

sorry. my accounts got mixed up, that was me, the previous post

@lesnitsky
Copy link
Contributor

@kevmoo is there any roadmap for googleapis.dart? It's been a while since this issue was raised and I'm now facing the same problem.

If that's not something that is planned or will be worked on, it's worth dropping the method from the public API.

@lesnitsky
Copy link
Contributor

It also seems that even if the underlying request wasn't returning error, streaming wouldn't be possible still, since tail() returns TailLogEntriesResponse, as opposed to Stream<TailLogEntriesResponse>.

@kevmoo
Copy link
Collaborator

kevmoo commented Mar 5, 2024

I'm downstream of things that need to be fixed. I'd happy accept a pull request excluding these APIs.

@lesnitsky
Copy link
Contributor

lesnitsky commented Mar 5, 2024

I went through the steps described under contributing section of the readme and it's not entirely clear to me if there is some "exclusion" mechanism. If I just remove the tail() method and related classes – those will be back after someone runs run_config generate again. Is my understanding correct?

@kevmoo
Copy link
Collaborator

kevmoo commented Mar 5, 2024

You have to add a delta file. See https://github.com/google/googleapis.dart/tree/master/overrides

@lesnitsky
Copy link
Contributor

I can't find any notes on how to name those diffs. Should those just be unique or is there any relation to commit shas?

@kevmoo
Copy link
Collaborator

kevmoo commented Mar 5, 2024 via email

@lesnitsky
Copy link
Contributor

I've also tried calling the API directly with a raw HttpClient and got the same error response. Could a relevant issue be raised internally?

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

No branches or pull requests

4 participants