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

email reader done. #113

Merged
merged 9 commits into from
Apr 30, 2024
Merged

email reader done. #113

merged 9 commits into from
Apr 30, 2024

Conversation

fdddf
Copy link
Contributor

@fdddf fdddf commented Apr 22, 2024

#70 提交代码

Copy link
Contributor

@JiazhenBao
Copy link
Collaborator

功能审核中

Copy link
Contributor

@JiazhenBao
Copy link
Collaborator

Copy link
Contributor

@fdddf
Copy link
Contributor Author

fdddf commented Apr 24, 2024

功能审核反馈:https://approve-community1.app.codewave.163.com:443/upload/app/EmailFetcher_fdddf退回原因_20240424174310365.docx

那两个bug是同一个问题,已经修改了。

1、获取邮件时,建议将文件存储到云存储空间,并返回云存储的文件url。
这个有其它依赖库的需求,这里就不实现了。

2、如果获取失败后,重新拉取的邮件是否有做去重
IMAP的支持

3、获取的时候是否可以通过关键词过滤,这类扩展能不能有没有支持
已经支持,但是有些邮件服务器可能不支持;

4、获取邮件分手动和自动,作为通用能力,建议都支持
平台服务端逻辑有定时功能,可以设置后自动获取。

@JiazhenBao
Copy link
Collaborator

第二点没有理解,麻烦解释一下。
第一点我们的预期是希望有附件下载功能的,不支持附件下载的话,会在功能完整度上扣分。麻烦确认一下是否提交当前内容。

@fdddf
Copy link
Contributor Author

fdddf commented Apr 25, 2024

第二点没有理解,麻烦解释一下。 第一点我们的预期是希望有附件下载功能的,不支持附件下载的话,会在功能完整度上扣分。麻烦确认一下是否提交当前内容。

IMAP协议因为支持获取messageid, 一个批次中如果有一条失败了会标记重试。

需求描述太简单了, 附件下载我再看下加上。

Copy link
Contributor

@fdddf
Copy link
Contributor Author

fdddf commented Apr 27, 2024

附件下载功能加上了。

@JiazhenBao
Copy link
Collaborator

@fdddf
Copy link
Contributor Author

fdddf commented Apr 28, 2024

仍然存在问题,见附件 https://approve-community1.app.codewave.163.com:443/upload/app/EmailFetcher_fdddf退回原因0428_20240428162946056.docx

报错信息日志是啥;
确认下版本是不是1.1.2 , 另外应用是否配置了阿里云OSS
javax.mail这个库确实有获 取不到其它文件夹的邮件的情况,这个我再确认下。

Copy link
Contributor

@JiazhenBao
Copy link
Collaborator

CR反馈:

  1. 当前邮箱只能应用配置。无法在运行时状态新增邮箱。
  2. 文件存储协议建议与主项目一致,这个jar兼容oos协议。
com.amazonaws aws-java-sdk-s3 1.12.440

@JiazhenBao
Copy link
Collaborator

  1. 使用 @NaslConfiguration注解没有注释,导致ide里面也没用对应的注释

Copy link
Contributor

@fdddf
Copy link
Contributor Author

fdddf commented Apr 29, 2024

  1. 使用 @NaslConfiguration注解没有注释,导致ide里面也没用对应的注释

都已经修改好了。

@JiazhenBao
Copy link
Collaborator

抛出的异常需要是RuntimeException的子类,用RuntimeException无法在ide中处理

Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

@JiazhenBao JiazhenBao merged commit 123a193 into netease-lcap:main Apr 30, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants