fix(deps): update dependency @strapi/strapi to v4.13.1 [security] #14

plural-renovate · 2023-04-19T23:56:50Z

This PR contains the following updates:

Package	Type	Update	Change
@strapi/strapi (source)	dependencies	minor	`4.5.2` -> `4.13.1`

GitHub Vulnerability Alerts

CVE-2023-22894

Summary

Strapi through 4.7.1 allows unauthenticated attackers to discover sensitive user details for Strapi administrators and API users.

Details

Strapi through 4.7.1 allows unauthenticated attackers to discover sensitive user details for Strapi administrators and API users. The unauthenticated attacker can filter users by columns that contain sensitive information and infer the values by the changes in the API responses. An unauthenticated attacker can exploit this vulnerability to hijack Strapi administrator accounts and gain unauthorized Strapi Super Administrator access by leaking the password reset token and changing the admin password. This can be exploited on all Strapi versions <=4.7.1.

IoC

The exploitation of CVE-2023-22894 is easily detectable, since the payload is within the GET parameters and are normally included in request logs. The following regex pattern will extract requests that are exploiting this vulnerability to leak user's email, password and password reset token columns.

You can search log files for this IoC by using the following grep command.

If the above regex pattern matches any lines in your log files, take extra precaution to look out for multiple requests that include password, reset_password_token or resetPasswordToken. This would indicate that an attacker has leaked the password hashes and reset tokens on your Strapi server and you need to immediately start an incident response!

Impact

All Strapi users below 4.8.0

CVE-2023-34093

Summary

Anyone (Strapi developers, users, plugins) can make every attribute of a Content-Type public without knowing it.

Details

When dealing with content-types inside a Strapi instance, we can extend those using the appropriate container:

strapi.container.get('content-types').extend(contentTypeUID, (contentType) => newContentType);

The vulnerability only affects the handling of content types by Strapi, not the actual content types themselves. Users can use plugins or modify their own content types without realizing that the privateAttributes getter is being removed, which can result in any attribute becoming public. This can lead to sensitive information being exposed or the entire system being taken control of by an attacker(having access to password hashes).

PoC

Extend any content type on runtime (like in the bootstrap functions) and do a copy of the content-type object.

strapi.container.get('content-types').extend(contentTypeUID, (contentType) => {
  const newCT = { ... contentType, attributes: { ...contentType.attributes, newAttr: {} } };
  return newCT;
});

This will have as effect to remove the getter and as we rely on it in sanitization, every attributes will be considered as public.

Impact

Everyone can be impacted. Depending on how people are using/extending content-types. If the users are mutating the content-type, they will not be affected.

CVE-2023-39345

System Details

Name	Value
OS	Windows 11
Version	4.11.1 (node v16.14.2)
Database	mysql

Description

I marked some fields as private fields in user content-type, and tried to register as a new user via api, at the same time I added content to fill the private fields and sent a post request, and as you can see from the images below, I can write to the private fields.

To prevent this, I went to the extension area and tried to extend the register method, for this I wanted to do it using the sanitizeInput function that I know in the source codes of the strap. But the sanitizeInput function did not filter out private fields.

  const { auth } = ctx.state;
  const data = ctx.request.body;
  const userSchema = strapi.getModel("plugin::users-permissions.user");

  sanitize.contentAPI.input(data, userSchema, { auth });

here's the solution I've temporarily kept to myself, code snippet

  const body = ctx.request.body;

  const { attributes } = strapi.getModel("plugin::users-permissions.user");

  const sanitizedData = _.omitBy(body, (data, key) => {
    const attribute = attributes[key];

    if (_.isNil(attribute)) {
      return false;
    }

    //? If you want, you can throw an error for fields that we did not expect.

    // if (_.isNil(attribute))
    //   throw new ApplicationError(`Unexpected value ${key}`);

    // if private value is true, we do not want to send it to the database.
    return attribute.private;
  });

  return sanitizedData;

Release Notes

strapi/strapi (@strapi/strapi)

fix(deps): update dependency @strapi/strapi to v4.13.1 [security] #14

Are you sure you want to change the base?

fix(deps): update dependency @strapi/strapi to v4.13.1 [security] #14

Conversation

plural-renovate bot commented Apr 19, 2023 • edited Loading

GitHub Vulnerability Alerts

Summary

Details

IoC

Impact

Summary

Details

PoC

Impact

System Details

Description

Release Notes

⚠️ Breaking Changes ⚠️

⚠️ Security Warning and Notice ⚠️

💅 Enhancement

🚀 New feature

🔥 Bug fix

🚨 Security

⚙️ Chore

📚 Update and Migration Guides

⚙️ Chore

📖 Documentation

📚 Update and Migration Guides

🔥 Bug fix

⚙️ Chore

💅 Enhancement

📚 Update and Migration Guides

⚙️ Chore

🔥 Bug fix

📖 Documentation

📚 Update and Migration Guides

⚠️ Security Warning and Notice ⚠️

⚙️ Chore

💅 Enhancement

🔥 Bug fix

📚 Update and Migration Guides

🚀 New feature

⚙️ Chore

💅 Enhancement

🔥 Bug fix

📚 Update and Migration Guides

🔥 Bug fix

⚙️ Chore

📚 Update and Migration Guides

⚙️ Chore

💅 Enhancement

🔥 Bug fix

📚 Update and Migration Guides

⚙️ Chore

🔥 Bug fix

📖 Documentation

📚 Update and Migration Guides

💅 Enhancement

🔥 Bug fix

⚙️ Chore

📚 Update and Migration Guides

🔥 Bug fix

Configuration

plural-renovate bot commented Apr 19, 2023 •

edited

Loading