Identity Users with extra properties and Maximum call stack size exceeded #3318 | Support

AlderCove created 2 years ago

ABP Framework version: v5.1.4 & 5.2.2
UI type: Angular
DB provider: EF Core
Tiered (MVC) or Identity Server Separated (Angular): yes
Exception message and stack trace: ERROR RangeError: Maximum call stack size exceeded
Steps to reproduce the issue:
1. Create new project
1. Follow ABP Framework Quick example for Module Extension System (https://docs.abp.io/en/abp/5.2/Module-Entity-Extensions#quick-example) to add SSN to YourProjectNameModuleExtensionConfigurator class inside the Domain.Shared project and to add column to table (https://docs.abp.io/en/abp/5.2/Module-Entity-Extensions#database-mapping)
1. Add migration and update database
1. Run the projects (API, Identity Server and Angular Apps)
1. Login to Angular App
1. Add 10-12 Users
1. Refresh the page
1. Observe the error

Hi,

I encountered this issue while adding several properties to the IdentityUser entity and Dtos (v5.1.4) . In my application, this error prevents the Users page from fully loading - some of the columns in the last few rows aren't rendered including the Action menu button.

I was able to reproduce the error with a scratch project (v 5.2.2), by adding a single property to the IdentityUser entity. I followed the Architecture -> Modularity -> Customizing/Extending Modules -> Module Entity Extension System of the documentation for 5.2 to add the SSN to the IdentityUser. I needed to create 12 or so users before the error cropped up:

When I remove the configuration for the extra property from the PortalModuleExtensionConfigurator in the Domain.Shared project, and restart the identity server & API, the error goes away:

        * See the documentation for more:
         * https://docs.abp.io/en/abp/latest/Module-Entity-Extensions
         */
        //ObjectExtensionManager.Instance.Modules()
        //.ConfigureIdentity(identity =>
        //{
        //    identity.ConfigureUser(user =>
        //    {
        //        user.AddOrUpdateProperty<string>( //property type: string
        //            "SocialSecurityNumber", //property name
        //            property =>
        //            {

        //                //validation rules
        //                property.Attributes.Add(new RequiredAttribute());
        //                property.Attributes.Add(new StringLengthAttribute(64) { MinimumLength = 4 });
        //            });
        //    });
        //});

Appreciate your help.

Thanks Jamie

4 Answer(s)

0

liangshiwei created 2 years ago
Support Team Fullstack Developer

Hi,

We created an internal issue for this and will check it.
0

AlderCove created 2 years ago

Thanks.

I just want to highlight that this is an urgent issue for us, as we are going live very soon.

The impact of this issue in our system is that the User page is not fully loading. Sometimes the columns are not fully rendered, including the action menu button:

Here are a couple of examples:

muhammedaltug created 2 years ago

Hello,

The fix for this issue will be available for the next 5.3 patch version. Currently, we don't have plans to publish a new version for version 5.2.

But there is a workaround below for this bug;

// app.component.ts
import { PermissionDirective } from '@abp/ng.core';
import { Component } from '@angular/core';
import { distinctUntilChanged } from 'rxjs/operators';
@Component({
  selector: 'app-root',
  template: `
    &lt;abp-loader-bar&gt;&lt;/abp-loader-bar&gt;
    &lt;abp-dynamic-layout&gt;&lt;/abp-dynamic-layout&gt;
  `,
})
export class AppComponent {}

function checkCustom() {
  if (this.subscription) {
    this.subscription.unsubscribe();
  }

  this.subscription = this.permissionService
    .getGrantedPolicy$(this.condition || '')
    .pipe(distinctUntilChanged())
    .subscribe(isGranted => {
      this.vcRef.clear();
      if (isGranted) this.vcRef.createEmbeddedView(this.templateRef);
      this.cdRef.markForCheck();
    });
}
PermissionDirective.prototype['check'] = checkCustom;

0

AlderCove created 2 years ago

that worked - thank you.

Have an answer to this question? Log in and write your answer.

Assignee

muhammedaltug

Labels