This allows to specify aiohttp.Application kws: see https://docs.aiohttp.org/en/stable/web_reference.html#aiohttp.web.Application

E.g.: for middlewares

        cirrina.Server(app_kws={
            'middlewares': [error_middleware]
        })

Does the javascript websocket client implementation really belong to the package content? It's just another implementation of a client which is able to talk to a cirrina server. The examples/documentation might be a better place.

Note: work in progress ... do NOT merge

• [x] Register HTTP routes via decorators
• [x] Register websocket callbacks via decorators
• [x] Enable websockets and JRPC

• [ ] Pass context object

(1) Register HTTP routes via decorators

@app.get('/')
async def index():
    pass

(2) Register websocket callbacks via decorators

@app.websocket_connect
async def on_websocket_connect():
    pass

(3) Enable websockets and JRPC

app.enable_websockets()
app.enable_rpc('/jrpc')

(4) Easy setup of cirrina app

app = cirrina.Server()

@app.get('/')
async def index():
    pass

if __name__ == '__main__':
    app.run()

When shutting down cirrina while using Python 3.9, I get this error:

^CTraceback (most recent call last):
...
  File "/usr/lib/python3/dist-packages/cirrina/server.py", line 183, in run
    for task in asyncio.Task.all_tasks():
AttributeError: type object '_asyncio.Task' has no attribute 'all_tasks'

The problem is that asyncio.Task.all_tasks was deprecated in py3.7 and removed in py3.9: https://docs.python.org/3.8/library/asyncio-task.html#asyncio.Task.all_tasks

This patch permits to use aiohttp idiomatic use of exceptions for control flow (e.g. raise aiohttp.web.HTTPFound('/redirect')).

Also, if creating and tearing down the session context fails, it's best to just fail with a 500 error rather then continuing with the request: so, don't try/catch errors there either.

Python's exception chaining will still preserve the original error (if any), in case an error is raised in the cleanup (finally) branch.

Fix #27

The session wrapper swallows any exception, therefore preventing aiohttp idiomatic use of HTTPExceptions for control flow.

For example, this handler wouldn't work (see https://docs.aiohttp.org/en/stable/web_quickstart.html#exceptions):

async def handler(request):
    raise aiohttp.web.HTTPFound('/redirect')

because the exception would be caught in the above "session wrapper" except clauses and never re-raised.

When running cirrina, I got these deprecation warnings:

tests/test_remotescreen.py::RemoteScreenTest::test_query
  /home/users/bollal/.virtualenvs/rlx-web/lib/python3.7/site-packages/aiohttp/web_urldispatcher.py:188: DeprecationWarning: Bare functions are deprecated, use async ones
    "Bare functions are deprecated, " "use async ones", DeprecationWarning

tests/test_app.py::AppTest::test_logout
  /home/users/bollal/src/github.com/neolynx/cirrina/cirrina/server.py:305: DeprecationWarning: The 'warn' method is deprecated, use 'warning' instead
    self.logger.warn('No valid session in request for logout')

This PR fixes both.

Hi @neolynx,

To enable cirrina in https://github.com/the-benchmarker/web-frameworks/issues/1638, I need to make it runnable on python 3.8.

This PR :

• [x] Add classifiers to setup.py so it could be installable in python >= 3.5
• [x] Add python >= 3.5 on CI
• [x] Fix CI to push on pypi

Regards,

When uploading files, it's possible that "recived_filename" is not yet defined when it's used. See: https://github.com/neolynx/cirrina/blob/8c69c0ea4e4a03ce4f9f91ecda48d16bd832ab7f/cirrina/server.py#L458

E.g. error log:

local variable 'received_filename' referenced before assignment
                                               Traceback (most recent call last):
                                                 File "/opt/roche/home/cirrina/cirrina/server.py", line 356, in _wrap
                                                   ret = (await func(request))
                                                 File "/opt/roche/home/cirrina/cirrina/server.py", line 458, in upload_handler
                                                   filename = received_filename.replace("\"", "")
                                               UnboundLocalError: local variable 'received_filename' referenced before assignment

General cleanup and update. In the last 7 months aiohttp has grown quite a lot.

- Fixed PEP8
- Removed lots of duplicate code
- Updated to aiohttp 2.2
- Removed backwards compatibility pre-python3.5
  to use aiohttp's async with functionality and async def, much clearer syntax
- Using new aiohttp startup methods and signals
- Added TODO entries regarding login methods
- Removed old-and-unused debian files
- Fixed client style
- Moved documentation outside, deprecated websocket client library

I'm planning on adding individual websocket handling (by session or by a self-assigned id in authless use cases) and an aiohttp websocket client example soon. This PR lacks a lot of testing, does major changes and breaks things.

I've not added tests nor sphinx doc because I first wanted to see your opinion on the current PR, @neolynx