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

Db update OpenIddictApplications.LogoUri = "dummy_string or http://path_to_online_logo" then Auth Server Raise 500 exeption on login form component #16804

Closed
1 task done
hungtrinh opened this issue Jun 10, 2023 · 1 comment
Assignees

Comments

@hungtrinh
Copy link

hungtrinh commented Jun 10, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Description

On db, table OpenIddictApplications if exits an record with LogoUri = 'dummy_string_not_start_by_char_/" or 'http://real_online_logo' Then access Auth Server will rasie error 500 on login form component

Reproduction Steps

After login success, on Angular web screen: '/openiddict/Applications' (Administration > OpenId > Applications)

  1. Click Action / Edit an Record
  2. On Screen Edit Application, Set Logo Uri field input: abcdef or http://online_logo_path
  3. On Screen Edit Application, Click Save

Expected behavior

After Set LogoUri = 'dummy text, or http://online_logo_path' then auth server (local: https://localhost:44334/) working normal

Actual behavior

On Auth server, login component section show error message 500 Internal Server ErrorAn internal error occurred during your request!

Here is log on Auth Server

authserver    | [15:44:34 ERR] An unhandled exception has occurred while executing the request.
authserver    | System.ArgumentException: The path in 'value' must start with '/'. (Parameter 'value')
authserver    |    at Microsoft.AspNetCore.Http.PathString..ctor(String value)
authserver    |    at Microsoft.AspNetCore.Mvc.Routing.UrlHelperBase.Content(HttpContext httpContext, String contentPath)
authserver    |    at AspNetCoreGeneratedDocument.Pages_Index.<>c__DisplayClass29_1.<<ExecuteAsync>b__18>d.MoveNext() in /home/runner/work/abp/abp/aspnet-core/src/AuthServer/Pages/Index.cshtml:line 108

aspnet-core/src/AuthServer/Pages/Index.cshtml:line 108 FILE CONTENT

105      @if (!application.LogoUri.IsNullOrEmpty())
106      {
107          <div class="mx-auto">
108              <img src="@Url.Content(application.LogoUri.EnsureStartsWith('~'))" style="height:64px" class="mb-3" />
109          </div>
110      }

Regression?

No response

Known Workarounds

No response

Version

7.2.2

User Interface

Common (Default)

Database Provider

EF Core (Default)

Tiered or separate authentication server

Separate Auth Server

Operation System

Windows (Default)

Other information

No response

@hungtrinh hungtrinh added the bug label Jun 10, 2023
@hungtrinh hungtrinh changed the title Db update OpenIddictApplications.LogoUri = "dummy_string or http://path_to_online_logo" then Server Raise 500 exeption Db update OpenIddictApplications.LogoUri = "dummy_string or http://path_to_online_logo" then Auth Server Raise 500 exeption on login form component Jun 10, 2023
@maliming maliming self-assigned this Jun 11, 2023
@maliming
Copy link
Member

hi

We will fix it in the next patch version.

Thanks.

@maliming maliming removed the bug label Jun 11, 2023
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

2 participants