should we support making this configurable?