Skip to content

Implement plugin manager #85

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
Open
Lycopersicum wants to merge 41 commits into
base: master
Choose a base branch
from
Open

Implement plugin manager #85

Lycopersicum wants to merge 41 commits into from

Conversation

@Lycopersicum
Copy link

@Lycopersicum Lycopersicum commented Oct 19, 2018

No description provided.

@codecov
Copy link

codecov bot commented Oct 19, 2018 •
edited
Loading

Codecov Report

Merging #85 into master will decrease coverage by 2.87%.
The diff coverage is 66.01%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #85      +/-   ##
==========================================
- Coverage   82.73%   79.85%   -2.88%     
==========================================
  Files          14       27      +13     
  Lines        1402     1708     +306     
==========================================
+ Hits         1160     1364     +204     
- Misses        242      344     +102
Flag Coverage Δ
#plugins 27.81% <66.01%> (?)
#regular 51.52% <16.66%> (-7.61%) ⬇️
#secure 37.23% <16.66%> (-4.49%) ⬇️
Impacted Files Coverage Δ
examples/rest-server/settings.c 84.29% <100%> (+4.56%) ⬆️
examples/rest-server/restserver.c 78.81% <100%> (+2.31%) ⬆️
lwm2m-framework/include/lwm2m_framework.hpp 100% <100%> (ø)
http-framework/include/http_framework.hpp 100% <100%> (ø)
http-framework/include/callback_handler.hpp 100% <100%> (ø)
http-framework/include/response.hpp 100% <100%> (ø)
http-framework/include/request.hpp 100% <100%> (ø)
plugin-manager/include/plugin_manager_core.hpp 100% <100%> (ø)
plugin-manager/include/plugin_manager.hpp 100% <100%> (ø)
http-framework/src/ulfius_http_framework.cpp 38.29% <38.29%> (ø)
... and 19 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f532e21...97a9dd0. Read the comment docs.

Tomas Markevičius and others added 21 commits October 22, 2018 15:37
Clean up http framework, separate headers and source files
5f1f2d2
@taubel
Changed request and response object initialization for easier object …
25ed008 
…maintenance
Improve ulfius http framework names, clean up interfaces
436266c
Clean-up lwm2m framework interface, update basic lwm2m framework
c863153
Update plugins lifetime, change lwm2m context (&rest to rest.lwm2m)
e8980f0
Add test plugin to test GET, PUT & POST requests
8a17a54
Merge branch 'plugin-manager' of https://github.com/8devices/wakaama …
8c86225 
…into plugin-manager
Fix merge conflict
3efe2f5
Remove debug prints
1ce6554
Update test plugin response headers
6b36e7f
Add plugins manager coverage tests
9039298
Remove debug code fragment
22ee26e
Go to BASE_DIR in beggining of every function that depends on path
0bc231d
Fix tests (base dir now is full path), add plugins.cfg
8cfbf4f
Add more directory paths
2c38f26
Add done() calls to before and after sections
48fad35
Do not set status code multiple times
2aa67c2
Build plugins in separate directory instead main plugin directory
2e5aeb1
Do not return status code 500 on all errors
5a20a22
Copy server object instead of modifying original
4346945
Change mkdir -P flag to valid lowercase -p flag
781f3c9
Tomas Markevičius added 13 commits October 26, 2018 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Lycopersicum @taubel