Skip to content

Consider making Redis an optional API dependency #519

Description

@JonCook

Hi,

It seem Redis is now a mandatory dependency in order to run a helioviewer API? Reviewing index.php

$redis = new Redis();
$redis->connect(HV_REDIS_HOST,HV_REDIS_PORT);
$rateLimiter = new RateLimit\RedisRateLimiter($redis,$prefix);

It seems it always connects despite the value of HV_ENABLE_STATISTICS_COLLECTION and HV_ENFORCE_RATE_LIMIT

For providers who don't wish to enable stats collection and enforce rate limiting in this way - could it be possible to switch it off all together?

Thanks
Jon

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions