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

Change array merging #212

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

janmaennig
Copy link

Before, in multipart validation, the validator lost the text part with the request data. So all validations failed if I have a file and request data, for example as json.

…I can used files and text data in the same request part
* 'master' of github.com:thephpleague/openapi-psr7-validator: (47 commits)
  Drop test for valid epsilon
  Add use statement for round
  Use epsilon to compare floating points
  Fix type errors
  Unignore phpstan errors, fixed now. Add polyfill 8.0 for `str_starts_with`
  Fix tests for boolean validation
  Fix string-to-boolean casting
  Handle path parameters for arrays correctly
  Add getVerboseMessage to AddressValidationFailed exception
  Fix test notice
  Fix content type parameter comparison
  (thephpleague#134) Add support for comma-separated list of encoding content types
  Fix cs errors. Use PHP 7.2 as phpstan reference language version to avoid false positive errors
  PathFinder: Adding filtering of placeholder routes scored by matching parts count
  Also remove check from multi-part server request validation
  Use devizzent/cebe-php-openapi to be compatible with OpenApi 3.1
  Allow specifying encoding for optional parts of multipart data without validation errors
  Temporary silence phpstan for checking type of magic props
  Fix codestyle
  Access operation security element directly, avoiding getSerializableData()
  ...

# Conflicts:
#	src/PSR7/Validators/BodyValidator/MultipartValidator.php
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.

1 participant