DB table needs restructuring

  • I have encountered errors when parsing URL from warmup table swift_performance_warmup.

    The cause is that the database field url is VARCHAR(255) by default, which is not enough for long URLs, or if URL contains non-latin characters, as was in my case.

    Setting this field to VARCHAR(2048) solved the problem: now all URLs can be read properly and indexed, otherwise there was an error 404 – which is obvious, because by default MySQL server cuts the long URL up to field length limit without throwing notice or warning.

    Besides that, the plugin is great, well-written source code, all database queries use prepared statements, security practices are implemented well.

    So, SWIFT, please update the database structure in the next version!

    P.S.: Also, update simple HTML DOM parser to at least version 1.9.1, the used version 1.5 (even modified in-house) has bugs.

  • The topic ‘DB table needs restructuring’ is closed to new replies.