Continued:
- if software is (still) None, then it needs to be determined, not otherwise
  (logic error)
- commented out some DEBUG lines

Continued:
- added config key "rss_limit" (default: 50)

Continued:
- reformatted ({}) block
- commented out debug line
- converted print("", foo, bar) to masked (f) version
- fixed logic bug when not all needed steps were executed

Continued:
- always add non-existing instances, also when their software type is not
  detectable, e.g. not existing
- also skip recently checked instances

Continued:
- re-introduced wrapper function around each column of the "model" `instances`
- marked set_data() as private (now named _set_data())
- these wrappers do NOT invoke instances.update_data(), you have to do this
  from your code

Continued:
- first load main packages, then sub

Fixed:
- also the other way around

Continued:
- 'data' is optional, e.g. empty POST bodies are okay
- fixed handling of JSON replies from peertube instances:
- the request /api/v1/server/followers is correct, then always the JSON has
  "follow" and "following" as elements

Fixed:
- some instances have broken API at /api/v1/instance/domain_blocks which returns
  nodeinfo (see masto1.iana.de)

Continued:
- domain/path might be a valid installation path, but some bug in Friendica
  caused my instance to be reported as 'f.haeder.net/featured' which sucks
- still we need to let domain/path pass here as some installations might
  be happening to be installed in such a way
- sure, best option is always a sub-domain ...

Continued:
- added 'software' as optional parameter to instances.add()
- also prevent bad invocations when a link to a user profile has provided

Continued:
- removed some wrapper functions
- update instance data even when the connection didn't work or some error
  status code (>= 400) came

Fixed:
- ops, wrong way!

Continued:
- moved fba.locking to fba.helpers

Continued:
- moved blocks, instances to new package fba.models as these modules are only
  around a database table with the same name
- movved+renamed fba.log_error() to fba.models.error_log.add()
- pleroma has a /instance/about/index.html "API" which can be parsed for
  block lists if the JSON has returned no 'mrf_simple' and
  'quarantined_instances'

Continued:
- prevent .arpa domains being added by instances.add()

Continued:
- better query `last_error_details` as this column is always set to NULL on
  success

Continued:
- ops, don't double-fetch a single row, causing 2nd invocation of fetchone()
  to return None as the first one already fetched it

Fixed:
- one lesser return statement, currently I have no idea on how to rewrite this?

Continued:
- no more 'unknown' blocking reasons, just keep them empty

Continued:
- duplicated some code for when no "mrf_simple" is returned but only
  "quarantined_instances"

Continued:
- ValueException is wrong, it is ValueError

Continued:
- better checking with .endswith() instead of .split()[-1]
- don't crawl '.tld' domains

Continued:
- fixed finding "powered by " (start of string)

Continued:
- alias 'nextcloud social' to 'nextcloud'

Continued:
- log also domain,origin,blocker

Continued:
- empty string changed to None

Fixed:
- 'redefined-outer-name' lint fixed
- 'bad-indentation ' lint fixed

Continued:
- tidyup + validate domains/reasons

Continued:
- introduced fba.process_domain() for generic domain processing, please provide
  origin and command

Continued:
- tpzo fixed in function name
- also re-check with domain if hash wasn't found
- 'runtube.re' aliased to 'peertube'

Continued:
- renamed hash -> blocked_hash to avoid redefining builtin references
- also combined 2 isinstance() into one

Continued:
- introduced instances.deobsure() for simple deobscuring attempts, e.g. when
  they use * or ? when they don't want to show the domain, you can still
  deobsfucate it with the hash (some provide it) or obscured domain itself

Continued:
- also update last_instance_fetch when software type isn't determined yet
- include command name in log messages

Continued:
- moved fba.cache to fba.helpers
- introduced instances.is_recent() so importing long blocklists won't take so
  long

Continued:
- added optional --domain parameter to fetch only from one oliphant blocklist
  files

WIP:
- rewrote from out-dated federater blocklist to multiple up-to-date oliphant
  blocklists
- also prepared for local file support (documentation prepared)

Continued:
- moved version-related stuff to own helpers module

Continued:
- let's collect them all and handle them together, no "generic-catch-all" as
  this would hide actual bugs

Continued:
- avoid local variables

Continued:
- some Mastodon API responses have no comment in it

Continued:
- DEBUG level is enough here, avoids double-logging

Continued:
- also cut off trailing double-dot (some people cannot format their data in
  tables properly)
- a wildcat might be at the start of the string, so start checking there for it

Continued:
- whole input tag, no matter for what purpose is styled now
- sorted CSS members

Continued:
- logic fixed
- debug messages added

Continued:
- need to catch exceptions here so more instances can be added with
  software=None

Continued:
- reason['reason'] had been handled to tidyup.reason() which caused an error
- it needed to be flexible "detected": str, dict are only supported types for now

Continued:
- fixed handling of 404 (no exception) case when nodeinfo wasn't found
- copied (WIP!) fetching /about page to pleroma
- also added /about to /about/more as possible pages for misskey

Continued:
- .arpa domains are reverse-ip-address domains and should not be used generally
- please choose a proper domain name, e.g. lemmy.myhomepage.tld

Continued:
- always transport causing exception
- some places may still handle it

Rewrites:
- rewrote api.api_top() to only 2 parameters, 'mode' and 'amount'
- also remove duplicate `if amount > 500` lines and combined them into one

Continued:
- disabled 'duplicate-code' lint check as it found checking parameters as
  duplicated code while it is needed for each function

Fixes:
- some misskey installations don't return isSuspended/isBlocked fields

Continued:
- removed another broad except block
- added missing variable initialization
- there is no 'domain', aka. instance here
- fixed bare-except, you can use 'if foo in bar' instead

Continued:
- handle typical network exceptions here, too
- don't sys.exit() here anymore, exit function properly
- don't 'break' here, there was no loop, better return proper error code
- fixed a few problems, found by pylint

Continued:
- fetch[_wellknown]_nodeinfo() should return same dict with at least
  'status_code' and 'json' on success and 'error_message' on failure

Continued:
- if dict 'error' cannot be handled (their keys) then raise an exception
- put all type(foo) into single quotes
- commented out debug line

Continued:
- also need to put a try/except block around csrf.determine() invocations

Continued:
- forgot to copy-paste error_message check for "blocked"
- also needed to take fetched['json'] for fetched rows

Continued:
- some misskey (Chinese) instance returned broken UTF-8, so I had to catch
  UnicodeEncodeError, too

Continued:
- both ['error_message'] and ['json']['federated_instances'] is extremely rare

Continued:
- at least show the exception name

Continued:
- ops, don't copy whole HTML to last_error_message
- also need to initialize local variable 'peers' first

Continued:
- no parameter for 'timeout' (here: 'headers' wasn't handled over)

WIP(?):
- moved csrf.determine() of of central functions, it was causing to much CSRF
  checks and slowed down all

Continued:
- ops wrong fix

Continued:
- DEBUG level + commented out is enough here

Continued:
- caught more specific exception and not broad BaseException (still some are
  left)
- renamed variable blocks -> blocking to avoid confusion with imported
  fba.blocks

Continued:
- also here need to handle error_message
- fetched['json'] does now contain the rows

Continued:
- whyever, but a list is returned, not a dict

Continued:
- also catch this exception here, avoiding a lot of more requests to unreachable websites

Continued:
- data['json'] now contains the reply

Continued:
- still a requests.model.Response might be handled over

WIP:
- introduced network.fetch_api_url() which takes a whole URL (not domain/path
  separated) and returns it's reply in a json_reply dictionary as both other
  API functions are handling it

WIP:
- more refacturing away from "generic except blocks"

Continued:
- introduced get_json_api() which gives back a dictionary,
- 'status_code' in it is always set, so better test on 'error_message'

Fixed:
- had to add a 'import reqto' only here to make Python happy
- looks like something in Python internal mapping or so is broken?

Continued:
- first 'rows' was re-assigned with new values, which is typical for Python
- yet still this hasn't fixed the recursion error

Continued:
- removed redundant initialization of 'blockdict'
- removed local variable 'csrf'

Continued:
- updated execution in systemd service file

Continued:
- explained why the original website isn't reflecting this code

Continued:
- updated README.md

Continued:
- removed remains of code around pending_errors dictionary
- rewrote it towards instances.update_last_error()
- also rewrote/removed some try/except blocks

Continued:
- markdown is another requirement

Continued:
- ops, double commented out
- also peertube may need debugging later on

Continued:
- renamed fba/federation/ -> fba/networks/ to make room for ...
- added fba.federation.py module which has generic functions for federation
- introduced fba/locking.py module
- renamed aqcuire_lock() -> aqcuire()
- added locking.release()
- introduced fba/helpers/dict.py and tidyup.py
- renamed tidyu_*() to tidyup.*()
- this all-in-all was an attempt to fix cyclic imports, still some are left

WIP:
- avoid these generic catches all together and try to catch them individually
  where it makes sense

Continued:
- not deleted here ...

Continued:
- formatted for shorter lines

Continued:
- ops, have forgotten to move this function to own module

Continued:
- renaming 'exception' to 'exc' didn't fix the problem, I thought here about
  overlapping/overwriting definitions
- but pylint3 would have it shown anyway

Continued:
- eventlet is nowhere used
- need to import jinja2 templating like this

Continued:
- fixed logger message

Continued:
- ignored more files (tracing, coverage)
- rewrote towards fba.csrf.determine() so it will always be determined, causes
  another request unfortunately
- or can we somehow save the cookie jar ?
- logging of errors can now be disabled for debug purposes

Continued:
- fixed more "Undefined variable"

Continued:
- fixed bad indenting
- renamed json -> rows to avoid redeclaration with module 'json'
- sorted import members (standard imports first)
- removed unused imports
- fixed "Redefining name 'headers' from outer scope"

Continued:
- renamed originator -> command

Continued:
- moved headers,api_headers to fba.network module
- fixed more references

Continued:
- the repository is more up-to-date, let's fetch this instead
- had to parse the markdown code back to HTML to be able to traverse with
  existing code over it

Continued:
- removed some redundant CSS
- also made both :hover color the same
- td's text alignment is now left