-
Notifications
You must be signed in to change notification settings - Fork 575
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
"Claim not found" error message when calling function app #258
Comments
Hi @rforster-dev See the list of possible V1 UIs: https://github.com/FBoucher/AzUrlShortener/blob/dev/src/adminTools/README.md You can find the details how to get the URLs here: Hope this helps. Otherwise please provide a screenshot when possible where you get the URL from. |
OK so i've been doing some reading, and it looks like the error message "Claim not found" comes from this piece of code:
Within this file: https://github.com/FBoucher/AzUrlShortener/blob/main/src/shortenerTools/Domain/Utility.cs Using the code from main-v1, this all works perfectly. I can't see any documentation on what's required here to get this check to pass? Maybe it's a documentation concern for V2 of the tool. Any clarification would be appreciated on how this is worked round in V2 of this tool. |
In V1 the Get Function Url shows me the complete URL including the code=... part (https://shortenertools....azurewebsites.net/api/UrlList?code=xxxx) and by the nature of V1 the URL doesn't require authentication. @rforster-dev did you upgrade from V1 to V2 or directly deployed V2 headless? Documentation for the upgrade #196 is WIP |
Thanks for the response. So when I logged the bug originally - I had use the V2 of this, which is where I was getting the issue as i've described. When I created a new 1, using V1 of the code (main-v1) I have no issues, so not migrating upwards in versions, doing clean installs. To me, it feels like V2 has an issue if you don't use the Blazor admin and simply deploy it headless, and the line in the code I mentioned is always assuming that;
|
@rforster-dev @fs366e2spm Good catch, the documentation needs to be updated. |
Hi, I get the same error when trying to call from a different function (in the same tenant). I do authenticate my call with a bearer token, but still no luck. I am using V2 (fresh install) thanks for your Help! |
An additional point we encountered is by default our AAD created an app registration only requesting the User.Read graph permissions. GivenName is not part of that. You can either add the "profile" graph API permission to get to GivenName, or change the function app code to use ClaimType.Name instead of ClaimType.GivenName - both worked for us. |
If anyone needs a headless version of the V2 shortener that doesn't have this issue (which is indeed related to AAD security) it can be easily removed to help you implement it and the great new features before tackling proper AAD auth (which is a little more complex to implement). You will of course need to maintain your own version of the shortener or we could make this configurable for future versions so you have the choice of auth 😄 |
This would be great to have in the documentation. I was trying to test this in Postman and was getting the same issues!!! Thank you |
The new version only offers redirects. |
I'm trying to setup the AzUrlShortener using Sharepoint to control the list, but when the flow is triggered, I'm seeing this error message:
When I go to the URL in the browser as well, it also shows the same error message.
I've tried with the V1 deployment (no frontend), and also the V2 setup (no frontend).
I've used the url with the "code=" parameter, using the function key, the master key (i've tried all keys!)
Let me know if you need more information for debugging this? Thanks!
┆Issue is synchronized with this Trello card by Unito
The text was updated successfully, but these errors were encountered: