kristen pol → credited steveworley → .
kristen pol → credited steveworley → .
Perfect thanks @acbramley — I'll just do a little bit of testing as well and we can merge this one. We're starting to get D11 images ready so this will be super helpful.
kristen pol → credited steveworley → .
kristen pol → credited steveworley → .
kristen pol → credited steveworley → .
kristen pol → credited steveworley → .
kristen pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
fubarhouse → credited steveworley → .
steveworley → created an issue.
steveworley → created an issue.
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
steveworley → made their first commit to this issue’s fork.
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Hey frob! Yeah — that's right, submission to a HTTP endpoint, either a service that is remotely managed like VirusTotal or a HTTP endpoint that you manage to validate files.
For the initial release of this module, the HTTP request and response are hard coded so it might be 100% compatible with the VirusTotal spec. You can customise the name of the file upload and the key which the module will look in from the response but if it expects are more complicated request/response schema we'll need to accommodate a more pluggable solution to support that backend properly.
I've created a MR to resolve this; https://git.drupalcode.org/project/clamav/-/merge_requests/7. The CKEditor plugin doesn't pass through the tmp path in the file object but we can access the same values from the request object.
steveworley → created an issue.
Just to add to this one, I haven't tested myself yet but it should be pushed into Quant as unpublished I will confirm that though as I would expect this to be expedited if that's not the case.
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
This issue looked to be caused by a race condition. The order of operations would mean that the database is not always available when Drupal calls TrafficRegistry::invalidateTag. This would result in the select query failing and resulting in being in a state where the table doesn't exist and Drupal not being able to recreate.
I've resolved the issue with:
https://github.com/quantcdn/drupal/pull/155/commits/4d87390da5fbc23c248a...
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Kristen Pol → credited steveworley → .
Tested, added a few minor changes to support this properly:
- When constructing the http_response invalidation object we add a property for force so as to not conflict with tag exclusions
- Moved getTypes() definition to the plugin
Kristen Pol → credited steveworley → .