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.

Sign up for GitHub

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

Jump to bottom

broker_url should be broker_host #15

Closed
maaikelimper opened this issue Jun 4, 2024 · 2 comments
Closed

broker_url should be broker_host #15

maaikelimper opened this issue Jun 4, 2024 · 2 comments
Assignees
@david-i-berry
Labels
invalid This doesn't seem right

Comments

@maaikelimper
Copy link
Collaborator

In the configuration of wis2-downloader a user has to define "broker_url", "broker_port" and "protocol"

URL would be defined including the port and protocol, for example: mqtts://globalbroker.meteo.fr:8883 , instead the broker_url should be broker_host in the way it is used in wis2-downloader ....
@maaikelimper maaikelimper added the invalid This doesn't seem right label Jun 4, 2024
david-i-berry added a commit that referenced this issue Jun 4, 2024
@david-i-berry
Updates to address
72adc2a 
- #15
- #14
- #13
@david-i-berry
Copy link
Member

Variables renamed to:

{
    ...
    "broker_hostname": "globalbroker.meteo.fr",
    "broker_password": "everyone",
    "broker_port": 443,
    "broker_protocol": "websockets",
    "broker_username": "everyone",
    ...
}

@david-i-berry david-i-berry mentioned this issue Jun 10, 2024
Merged
maaikelimper added a commit that referenced this issue Jun 12, 2024
@david-i-berry @maaikelimper
Refactor - non dev mode, disk space limits and file naming (#18)
2dfa7cc 
* Updates to address
- #15
- #14
- #13

* - Refactor of app.py allow running in none dev mode.
- Change to use get, post and delete to manage subscriptions
- addition of openapi doc

* Addition of swagger template to render openapi doc.
Update of README.md

* conditional tls-set

* Update of readme. Update ot GET, POST and DELETE methods, e.g. POST content now via request body.

Improved error handling and response codes.

* Tweaks to readme.

* Tweaks to readme.

* Update validate_topic.py

print input-topic in case of validation error

* Option to disable topic validation added.

* Unlimited disk usage possible by setting limit to 0.

* Reversion to using filename from global cache.
Switch to using minimum free space rather than maximum download dir size.

* Update to readme to include validat_topics option.

* Update to config.json and swagger page to allow customisable base url.

* Log levels configurable.

---------

Co-authored-by: Maaike <mlimper@wmo.int>
@maaikelimper
Copy link
Collaborator Author

#18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@david-i-berry david-i-berry

Labels
invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@maaikelimper @david-i-berry