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

Enable API qso to accept also form data and not only Json #3115

Closed
wants to merge 19 commits into from

Conversation

emics
Copy link
Contributor

@emics emics commented May 21, 2024

Added the possibility of sending data to the "qso" method API also in www-x-form-urlencoded format to allow the automatic sending of QSOs via software such as Log4OM and other.
Perhaps it is not the most elegant way to develop this feature, you probably prefer to create a new method.
Obviously I leave the choice up to you.
73 emiliano

… www-x-form-urlencoded format to allow the automatic sending of QSOs via software such as Log4OM
@magicbug
Copy link
Owner

I can understand the appeal but prefer the current API methods as for Log4OM I believe it already supports Cloudlogs API :)

@emics
Copy link
Contributor Author

emics commented May 22, 2024

in reality Log4OM allows you to specify the URL of the POST call and a series of Key/Value pairs which however are sent with this syntax:
key1=value1&key2= value2&key3=value3

if I try to inject a json as a value obviously the API doesn't like it.

You choose how you think is best to proceed, in my local version I will fork the method for my use.

@emics
Copy link
Contributor Author

emics commented May 27, 2024

Hi @magicbug Please delete this pull Request, yesterday i have found the specific configuration in Log4OM for CloudLog. My changes are useless. sorry.

the changes in application/controllers/Api.php are useless.

@emics
Copy link
Contributor Author

emics commented Jun 5, 2024

no more needed

@emics emics closed this Jun 5, 2024
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.

2 participants