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

Invalid filter clause #749

Closed
frankpeng7 opened this issue Oct 22, 2020 · 7 comments
Closed

Invalid filter clause #749

frankpeng7 opened this issue Oct 22, 2020 · 7 comments
Assignees

Comments

@frankpeng7
Copy link

frankpeng7 commented Oct 22, 2020

Describe the bug
I was testing a new API. Somehow it worked in Postman but not in Graph explorer.

In Graph explorer, set the request url to:

GET https://graph.microsoft.com/beta/me/onlineMeetings?$filter=JoinWebUrl eq 'https://teams.microsoft.com/l/meetup-join/19%3ameeting_OTM3ODRlODctYjhiMS00MjZkLTkzODItNTk5Y2YyNGQwMzlj%40thread.v2/0?context={"Tid"%3a"909c6581-5130-43e9-88f3-fcb3582cde37"%2c"Oid"%3a"dc17674c-81d9-4adb-bfb2-8f6a442e4622"}'

got an error message saying "Invalid filter clause". However, if I copy-paste the same request url and access token to Postman, it worked as expected (see screenshots below).

Can you help identify if this is something wrong on Graph explorer's side?

Screenshots
Not working in Graph explorer:
image

Same request url and same token in Postman:
image

Desktop (please complete the following information):

  • OS: [Windows 10]
  • Browser [Edge]
  • Version [ 86.0.622.48]

Additional context
This is from internal. Feel free to reach out on Teams directly.

AB#7468

@ghost ghost added the ToTriage label Oct 22, 2020
@thewahome
Copy link
Collaborator

Hi @frankpeng7, this should be fixed. Are you still able to reproduce this?

@ddyett ddyett added the promote label Jan 11, 2021
@ezkemboi
Copy link
Contributor

ezkemboi commented Feb 2, 2021

It seems this is caused by white spaces in the URL.
As highlighted in #788 and with some fix in #827, this issue might be fixed.

@ghost ghost removed the ToTriage label Feb 8, 2021
@ddyett ddyett added Bug – P2 and removed promote labels Feb 8, 2021
@ddyett
Copy link

ddyett commented Feb 8, 2021

@ElinorW can you verify this is fixed and close.

@ElinorW
Copy link
Contributor

ElinorW commented Feb 10, 2021

@ezkemboi the spaces in between won't present an issue as they are encoded when the query is run.

@ezkemboi
Copy link
Contributor

@ElinorW that is true.
Thanks for clarification.

@ElinorW
Copy link
Contributor

ElinorW commented Feb 10, 2021

@frankpeng7
could you confirm if this is still an issue?
I have tried running a similar query and it works well.
Though I have observed getting the same error when I don't add the ' ' marks surrounding the URL or if I am missing a single one

@frankpeng7
Copy link
Author

Hi @ElinorW yes it's been working for a while. Looks like Graph explorer is now encoding the space.

@ElinorW ElinorW closed this as completed Feb 11, 2021
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

No branches or pull requests

5 participants