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

provide SmtpServer payload, fix content-type of VerifyUserEmailAddress… #44

Merged
merged 1 commit into from
May 7, 2021

Conversation

manuscrypt
Copy link

…Async

@manuscrypt manuscrypt changed the title provide SmtpServer payload, fix conent-type of VerifyUserEmailAddress… provide SmtpServer payload, fix content-type of VerifyUserEmailAddress… Mar 9, 2021
@psavva
Copy link

psavva commented Mar 29, 2021

I can confirm this exact same bug, I have just come across it now, developing against this library.
Please review, and commit the fix.

@psavva
Copy link

psavva commented Mar 29, 2021

@lvermeulen Any chance to review and have this part of the next release?

@psavva
Copy link

psavva commented Apr 14, 2021

@lvermeulen I hope you are keeping well?
Could you provide an update on merging this PR?

Best Regards
Panayiotis Savva

@lvermeulen lvermeulen merged commit e72b3ad into lvermeulen:main May 7, 2021
@lvermeulen
Copy link
Owner

Thanks @manuscrypt !

lvermeulen added a commit that referenced this pull request May 7, 2021
@psavva
Copy link

psavva commented Jun 27, 2022

@manuscrypt @lvermeulen I have used the latest version of the application, and am still hitting 400 HTTP error upon calling the function.

Have you managed to make this work?
Is there anything special we need to consider?

@psavva
Copy link

psavva commented Jun 27, 2022

Exception:
{"Call failed with status code 415 (Unsupported Media Type): PUT https://auth.ehealth4u.eu/auth/admin/realms/ehealth4u/users/295c4b83-9e83-4462-be78-0741a97a3af5/send-verify-email"}

Stack Trace:

   at Flurl.Http.FlurlRequest.<HandleExceptionAsync>d__23.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Flurl.Http.FlurlRequest.<SendAsync>d__19.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Flurl.Http.FlurlRequest.<SendAsync>d__19.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
   at Keycloak.Net.KeycloakClient.<VerifyUserEmailAddressAsync>d__312.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at Client.IdP.Keycloak.ClientKeycloak.<EmailVerification>d__10.MoveNext() in C:\repos\UCY\eHealth4u\eHealth4U.EHR.Backend\src\Client.IdP.Keycloak\ClientKeycloak.cs:line 237

@manuscrypt
Copy link
Author

@manuscrypt @lvermeulen I have used the latest version of the application, and am still hitting 400 HTTP error upon calling the function.

Have you managed to make this work? Is there anything special we need to consider?

Sorry, nothing special comes to mind. Can you get it working using postman (or the like)?
We are using jboss/keycloak:12.0.4

				{
					"name": "Send Verify Email",
					"event": [
						{
							"listen": "test",
							"script": {
								"exec": [
									""
								],
								"type": "text/javascript"
							}
						}
					],
					"request": {
						"auth": {
							"type": "bearer",
							"bearer": [
								{
									"key": "token",
									"value": "{{token}}",
									"type": "string"
								}
							]
						},
						"method": "PUT",
						"header": [],
						"url": {
							"raw": "{{keycloakApiHost}}/auth/admin/realms/{{realm}}/users/{{userId}}/send-verify-email?client_id=null&redirect_uri=null",
							"host": [
								"{{keycloakApiHost}}"
							],
							"path": [
								"auth",
								"admin",
								"realms",
								"{{realm}}",
								"users",
								"{{userId}}",
								"send-verify-email"
							],
							"query": [
								{
									"key": "client_id",
									"value": "null"
								},
								{
									"key": "redirect_uri",
									"value": "null"
								}
							]
						}
					},
					"response": []
				}

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

Successfully merging this pull request may close these issues.

3 participants