RubiconRouter _swap does not pass whole amount to RubiconMarket

BathToken LPs Unable To Receive Bonus Token Due To Lack Of Wallet Setter Method

Attacker Could Steal Almost All The Bonus Token In BathBuddy Vesting Wallet

First depositor can break minting of shares

Inconsistent Order Book Accounting When Working With Transfer-On-Fee or Deflationary Tokens

Cannot deposit to BathToken if token is Deflationary Token (BathHouse.sol)

previewWithdraw calculates shares wrongly

maxSellAllAmount and maxBuyAllAmount functions can be unintentionally paused (always revert).

Use `safeTransfer()`/`safeTransferFrom()` instead of `transfer()`/`transferFrom()`

RubiconRouter maxSellAllAmount does not trasnfer user's fund into its address, causing calls to always revert

Use `call()` instead of `transfer()` when transferring ETH in RubiconRouter