Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

a possible bug of FileBasedSessionManager #808

Closed
1 task done
leizhiyuan opened this issue Apr 16, 2019 · 0 comments · Fixed by #807
Closed
1 task done

a possible bug of FileBasedSessionManager #808

leizhiyuan opened this issue Apr 16, 2019 · 0 comments · Fixed by #807
Labels
type: bug Category issues or prs related to bug.

Comments

@leizhiyuan
Copy link
Contributor

leizhiyuan commented Apr 16, 2019

  • I have searched the issues of this repository and believe that this is not a duplicate.

Ⅰ. Issue Description

the full file name is not right.

Ⅱ. Describe what happened

   /**
     * Instantiates a new File based session manager.
     *
     * @param name                 the name
     * @param sessionStoreFilePath the session store file path
     * @throws IOException the io exception
     */
    public FileBasedSessionManager(String name, String sessionStoreFilePath) throws IOException {
        super(name);
        transactionStoreManager = new FileTransactionStoreManager(sessionStoreFilePath + name, this);
    }

the full name is not right

Ⅲ. Describe what you expected to happen

store {
  ## store mode: file、db
  mode = "file"

  ## file store
  file {
    dir = "root.data"

this should be dir, not the file name.

Ⅳ. How to reproduce it (as minimally and precisely as possible)

  1. xxx
  2. xxx
  3. xxx

Ⅴ. Anything else we need to know?

Ⅵ. Environment:

  • JDK version :
  • OS :
  • Others:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Category issues or prs related to bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants