search

rfidtool / ESP-RFID-Tool

A tool for logging data/testing devices with a Wiegand Interface. Can be used to create a portable RFID reader or installed directly into an existing installation. Provides access to a web based interface using WiFi in AP or Client mode. Will work with nearly all devices that contain a standard 5V Wiegand interface. Primary target group is 26-37bit HID Cards. Similar to the Tastic RFID Thief, Blekey, and ESPKey.
MIT License
496 stars 94 forks source link

Unable to contribute due to legacy toolchain 'DynamicJsonBuffer is a class from ArduinoJson 5' #19

Open nuschpl opened 5 months ago

nuschpl commented 5 months ago

Hello I've tried to follow the guide and add depdendencies:


    ArduinoJson library 5.13.1 by Benoit Blanchon
        https://github.com/bblanchon/ArduinoJson
    ESP8266FtpServer.h
        https://github.com/exploitagency/esp8266FTPServer/tree/feature/bbx10_speedup

Added the first lib directly from Arduino IDE, and the second via exported ZIP file of above URL. This resulted in compilation error:

In file included from /Users/User/src/ESP-RFID-Tool/Source Code/esprfidtool/esprfidtool.ino:605:
/Users/User/src/ESP-RFID-Tool/Source Code/esprfidtool/api.h:43:11: error: DynamicJsonBuffer is a class from ArduinoJson 5. Please see https://arduinojson.org/v7/upgrade-from-v5/ to learn how to upgrade to ArduinoJson 7
   43 |   DynamicJsonBuffer jsonAPIbuffer(bufferSize);

So my question is are those custom library variants really needed, can we make anything that software will be more maintanable ? I'm Arduino newbie but is there no way to add those dependencies as a reference to Sketch file instead of encouraging developer to manually do so ? This way this could could still be contributed